Disable KeyCloak impersonation

Please remove the impersonation role for all users.
Reasoning:
Impersonation as a concept is only appropriate for enterprise administrators to help with diagnosing issues for employees within their own enterprise. Even then, it would only be acceptable to be used sparingly and with significant oversight/audit (for example, this permission would have to be requested and approved by the CISO on a case-by-case basis).
With Keycloak, impersonation could likely be used to allow administrators to see a different citizen’s Health Information. There are no circumstances where this would be appropriate.

Realm admins can do this themselves.  Just remove the impersonation role from the realm-management role mappings.
Once removed, only the server admins can add impersonation back.

Who does "realm admins" consist of? The other ministry's clients who are building containers utilizing KeyCloak?

That would be each ministry's own development team, or the person who requested the realm to be created.

Is there a way to set this as default (and force it on existing) so as not leave it up to the realm admins since we have no way of enforcing it? The reason I ask is that I was told to submit the request here to get it done.

Why aren't you able to enforce it?  Realms are owned by ministry teams and they can make this change themselves.
Once the role is removed, the realm admins can't add it back again themselves.  Only the platform admin team can add it back.

I don't know why. I have no access to any of it as I don't work in dev. I'm simply security. All I know is I was asked to make the request by my management, and the devOps people told me to do that by submitting the ticket here. Can you do it?

I'm not part of the platform admin team, so I can only manage my own realms.
You need to find out who owns the realm on your devops team and have that person do what I said in my first reply.

Ok, Thanks. I'll take this all back to Shelly.

issue resolved: https://github.com/bcgov/ocp-sso/issues/74


Anerdw: Blacklist 52-800-953-0857

Anerdw requests the blacklist of the number 52-800-953-0857. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by VLAZ in Charcoal HQ


[Snyk] Security upgrade jsonwebtoken from 8.5.1 to 9.0.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

658/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-SEMVER-3247795
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

Checkmarx One – Scan Summary & Details – 003b45ad-cac5-4fbc-a120-152f01959c08
Fixed Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2022-23539
Npm-jsonwebtoken-8.5.1
Vulnerable Package

CVE-2022-23540
Npm-jsonwebtoken-8.5.1
Vulnerable Package

CVE-2022-23541
Npm-jsonwebtoken-8.5.1
Vulnerable Package


Add support for Python 3.10

Resolves #177

Bailing on this one for now, hitting several problems.


Installation issue: HA remotely accessible but can't install

Hi,
I followed instructions on installing the addon but I can't get past the message that my system would not be remotely accessible. As a matter of fact, I am configuring the addon remotely (from work).
Is it neccesary to have Home Assistant Cloud service (from Nabu Casa) or should "normal" remote access be sufficient?
Tried a number of restarts of the HA system but didn't help.
Best regards,
Björn

Hi @bjornvaessen
Home Assistant Cloud is not required. You can use any public domain you have configured.
What is important is :

Before you authenticate use the url https://my.home-assistant.io in the browser. When you hit that url if it doesn't know your ha instance public url it will reuqest you provide it.
The call back url configured in Strava is https://my.home-assistant.io

Hi Tim,
Thanks for your reply and help. Unfortunately, hitting the url https://my.home-assistant.io and setting it up correctly didn;t fix it... Still getting the message "Your Home Assistant Instance does not seem to have a public URL. The Strava Home Assistant integration requires a public URL" (after opening my HA url in the same browser tab)
Tried a re-install but also that didn't help.

Is your home assitant available on port 443 (https) on your public url or only the default 8123 port?


Add OAuth client to notebook template

Add an OAuth client that will get a token for opening pyspark contexts and opening pods for each kernel in the jupyter notebook namespace.

The oauth token is under /var/run/kuberenets.io/serviceaccount/token


FromSchema returns unknown (code example)

Code to reproduce:
import { JSONSchema, FromSchema } from 'json-schema-to-ts';

export const TestSchema: JSONSchema = {
	type: 'object',
	properties: {
		id: { type: 'number' },
	},
	additionalProperties: false,
} as const;

export type TestColumns = FromSchema<typeof TestSchema>;

// TestColumns is type `unknown`

TS version: 5.0.4
Library version: ^2.9.1
Also tried downgrading to 2.8.2
A bit sad as I was very excited to use this library! 🥺

I figured this out last night - It's because I typed TestSchema as JSONSchema instead of using satisfies.
Here is the working code:
import { JSONSchema, FromSchema } from 'json-schema-to-ts';

export const TestSchema = {
	type: 'object',
	properties: {
		id: { type: 'number' },
	},
	additionalProperties: false,
} as const satisfies JSONSchema;

export type TestColumns = FromSchema<typeof TestSchema>;

// TestColumns is type `{ id: number }`

Closing. I think this should help unblock somebody searching for "unknown" in the future. 👍

Nice !


Can this be used along with "laravelcollective/html": "5.2.*"

Can this be used along with "laravelcollective/html": "5.2.*" in Laravel 5?

Most likely yeah, it builds on a different form library though. I haven't tried myself so not sure what conflicts you might run into.


read magnetic card

Can you read magnetic card?
(bank card) note: just read, not scan

Sorry! I'm not sure what you mean. This works via NFC capability - bank cards are read via NFC so we can retrieve the number, cvc etc.


Adopt semantic versioning

Starting with version 2.0, we should explicitly adopt semantic versioning. This should be mentioned in the developer's documentation. All currently planned changes that break backwards compatibility should be added to the v.2.0 milestone.

As of https://github.com/mabuchilab/QNET/commit/372624eb67493d8df66ad6a9aef7e9dd302c82c2, semantic versioning is mentioned in the documentation


test: Update redhat.com IP address

<IP_ADDRESS> does not seem to be a valid IP address for redhat.com:
$ nslookup redhat.com
[...]
Non-authoritative answer:
Name:	redhat.com
Address: <IP_ADDRESS>
Name:	redhat.com
Address: <IP_ADDRESS>
This should fix some test failures.

/lgtm
/approve


Optimise UTxOIndex.difference.

Issue
https://input-output.atlassian.net/browse/ADP-2975.
Summary
This PR adjusts the UTxOIndex.difference function so that it does not reconstruct the entire index from scratch, and instead only deletes those entries that are necessary to delete.
This should be fast if the intersection of the two indices is relatively small w.r.t. to the size of the first index.
UTxOIndex.difference is called by UTxOSelection.fromIndexPair, which is called from balanceTransaction before it calls into the coin selection algorithm.

bors r+

bors r+

bors r+


php artisan acacia:install    Symfony\Component\Process\Exception\ProcessTimedOutException

PS C:\Devel_2022\Gpplus\Dev\savanna> php artisan acacia:install --force

Begin Installation     *

Publishing files
a. Publishing acacia configs
Publishing complete.
b. Publishing acacia core module
Copied Directory [\vendor\savannabits\acacia\publishes\acacia] To [\acacia]
Publishing complete.
c. Publishing laravel permission config and migration
Publishing complete.
d. Publishing laravel scout config
Publishing complete.
Dump autoload
Symfony\Component\Process\Exception\ProcessTimedOutException

The process "composer dump-autoload" exceeded the timeout of 60 seconds.
at C:\Devel_2022\Gpplus\Dev\savanna\vendor\symfony\process\Process.php:1152
1148▕
1149▕         if (null !== $this->timeout && $this->timeout < microtime(true) - $this->starttime) {
1150▕             $this->stop(0);
1151▕
➜ 1152▕             throw new ProcessTimedOutException($this, ProcessTimedOutException::TYPE_GENERAL);
1153▕         }
1154▕
1155▕         if (null !== $this->idleTimeout && $this->idleTimeout < microtime(true) - $this->lastOutputTime) {
1156▕             $this->stop(0);
1   C:\Devel_2022\Gpplus\Dev\savanna\vendor\symfony\process\Process.php:652
Symfony\Component\Process\Process::checkTimeout()
2   C:\Devel_2022\Gpplus\Dev\savanna\vendor\savannabits\acacia\src\Helpers\Helpers.php:23
Symfony\Component\Process\Process::getIterator()
PS C:\Devel_2022\Gpplus\Dev\savanna>

Hi @liteshare92,
I still haven't been able to reproduce this error. Would you please add the reproduction steps? Thanks

@liteshare92 this appears like an issue with your dev environment, although I could be wrong. I will proceed to close this issue. However, if the issue is still persisting, feel free to re-open the issue.

I finally encountered the same issue. The issue occurs when npm takes longer than (30 sec?) to install dependencies during installation. If this happens, just run npm install manually first so that the packages can be cached locally, then repeat php artisan acacia:install.


Extract NetworkProtection library to BSK

Please review the release process for BrowserServicesKit here.
Required:
Task/Issue URL: https://app.asana.com/0/0/1204296308622616/f
iOS PR: https://github.com/duckduckgo/iOS/pull/1813
macOS PR: https://github.com/duckduckgo/macos-browser/pull/1331
What kind of version bump will this require?: Minor
Optional:
Tech Design URL: https://app.asana.com/0/481882893211075/1204570769059898
CC: @diegoreymendez @mallexxx @samsymons
Description:
NetworkProtection is currently defined as a local package in the macOS repo. In order to enable code sharing with iOS, we’re moving it to a library in BSK.
Steps to test this PR:
Run these smoke tests:
https://app.asana.com/0/0/1203586333350365/f

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16
[ ] macOS 10.15
[ ] macOS 11
[x] macOS 12

Internal references:
Software Engineering Expectations
Technical Design Template

@graeme - One final note, I'm assuming the following code was added as part of this PR.  Since this PR only mentions moving files around it's slightly unexpected - I think it's worth considering to mention these kind of changes more explicitly in the description or inline comments to bring them into the scope of the PR.  My 2 cents.

--- a/Users/ddg/Developer/ddg/macos-browser/LocalPackages/NetworkProtection/Sources/NetworkProtection/KeyManagement/NetworkProtectionKeychainStore.swift
+++ b/Users/ddg/Developer/ddg/BrowserServicesKit/Sources/NetworkProtection/KeyManagement/NetworkProtectionKeychainStore.swift
@@ -90,7 +90,11 @@ final class NetworkProtectionKeychainStore {

     func deleteAll() throws {
         var query = defaultAttributes()
+#if os(macOS)
+        // This line causes the delete to error with status -50 on iOS. Needs investigation but, for now, just delete the first item
+        // https://app.asana.com/0/1203512625915051/1205009181378521
         query[kSecMatchLimit as String] = kSecMatchLimitAll
+#endif

         let status = SecItemDelete(query as CFDictionary)
         switch status {

Btw, I don't think I say this enough: but really nice work on this PR!!!  Thanks for doing this!
It was a breeze to review it by just diffing against develop.  Keep up the great work!


Set User object in alertmanager url.

Sets User object in alertmanager url to enable amtool to work with basic auth for use with proxies. Fixes #1028.

thanks for the contribution! looking at this code, it seems to me that we just want to copy the amURL we're getting from the config, and then update the path. Can you check to see if this code works for you?
diff --git a/cli/utils.go b/cli/utils.go
index 256c677..0828373 100644
--- a/cli/utils.go
+++ b/cli/utils.go
@@ -27,11 +27,9 @@ func (s ByAlphabetical) Less(i, j int) bool {
 }

 func GetAlertmanagerURL(p string) url.URL {
-	return url.URL{
-		Scheme: (*alertmanagerUrl).Scheme,
-		Host:   (*alertmanagerUrl).Host,
-		Path:   path.Join((*alertmanagerUrl).Path, p),
-	}
+	amURL := **alertmanagerUrl
+	amURL.Path = path.Join((*alertmanagerUrl).Path, p)
+	return amURL
 }

 // Parse a list of labels (cli arguments)

@stuartnelson3 I'm not familiar with the ** operator. Does that copy a struct?

alertmanagerUrl is a reference to a reference (**url.URL), so the double ** de-references to the actual struct and makes a copy. Check out the playground:
https://play.golang.org/p/MjVJheu_7vQ

@stuartnelson3 that code works for me, and makes sense to copy the whole thing instead of selected properties. I updated the PR.

cool. the code worked for me in my tests, can you double check that it works for you with your basic auth setup?

Yes, it works for me with basic auth


score demo

hello
i was testing the algorithm So i tried hello and hellô and the result was 0 wtf!

Can you send the command you tested?
You must have a fuzziness setting to accept mismatches.
[image: Inline image 1]
On Sat, Jul 1, 2017 at 1:52 PM, AMMISAID Mustapha<EMAIL_ADDRESS>wrote:

hello
i was testing the algorithm So i tried hello and hellô and the result was
0 wtf!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/joshaven/string_score/issues/32, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABVp4w0b_iNPr2NZLxzCryHdJ5xAGudks5sJpVegaJpZM4OLY8W
.

--
Sincerely,
Joshaven Potter
C:<PHONE_NUMBER>
Among the sins to which the human heart is prone, hardly any other is more
hateful to God than idolatry, for idolatry is at bottom a libel on His
character. The idolatrous heart assumes that God is other than He is - in
itself a monstrous sin - and substitutes for the true God one made after
its own likeness. Always this God will conform to the image of the one who
created it and will be base or pure, cruel or kind, according to the moral
state of the mind from which it emerges.  -- A. W. Tozer

Yes, it happened with me too.
var stringScore = require("string-score")

console.log(stringScore('1','1ó'))

It seems the algorithm confuses itself when adding extra characters at the end of the string.

The third argument is required for fuzzy matching. fuzzy means a degree of
matching when it isnt a match.
On Aug 31, 2017 11:12 PM, "Raphael D. Pinheiro"<EMAIL_ADDRESS>wrote:

Yes, it happened with me too.
var stringScore = require("string-score")
console.log(stringScore('1','1ó'))
It seems the algorithm confuses itself when adding extra characters at the
end of the string.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/joshaven/string_score/issues/32#issuecomment-326482630,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABVpzJWDvg5qd_MhOhTwne0nrTPu1Mqks5sd4QZgaJpZM4OLY8W
.

Ok, I didn't see it, thank you!


Missing options in ovpn_getclient.sh & ovpn_env.sh

Missing an option for NOT putting the ta.key in the ovpn file (no password option)
The following are part of ovpn_env.sh but have no reflection in ovpn_getclient.sh so do not end up in the resulting ovpn file
OVPN_CLIENT_TO_CLIENT
OVPN_DEVICEN
OVPN_DISABLE_PUSH_BLOCK_DNS
OVPN_DNS
OVPN_DNS_SERVERS
OVPN_EXTRA_SERVER_CONFIG
OVPN_FRAGMENT
OVPN_KEEPALIVE
OVPN_NAT
OVPN_PUSH
OVPN_ROUTES
OVPN_SERVER
OVPN_SERVER_URL

Can these be passed as environment variables, as well?


Add an example

https://github.com/symengine/symengine.py/blob/62a0d89b5b9ad00814c5cd2f72e697aa1c21dccd/symengine/tests/test_lambdify.py#L818-L824
Here's an example where you can use scipy's integrate with symengine providing a fast native compiled function instead of a python function. (Maybe we can compare a python function and symengine function)

cc @HaoZeke

cc @HaoZeke


Problem Pagination

Right now, NOJ's problem page pagination is such a mess that may not be maintainable. So we are about to rebuild it with Laravel's pagination, We've already done that at Contest Status Page, you can refer that when coding.

wanna be a contributor : )

Which pages need to be processed like this?

Finished.


nixos: fix systemd when systemd-boot is disabled

Description of changes
https://github.com/systemd/systemd/blob/11d5e2b5fbf9f6bfa5763fd45b56829ad4f0777f/units/meson.build#L317
various units are only present when systemd-boot is enabled, and nixos now fails to build if they are missing:
missing /nix/store/cz65qqqzjqkxl581xm502ylm15vmc5k0-systemd-powerpc64-unknown-linux-gnuabielfv2-256.2/example/systemd/system/systemd-hibernate-clear.service

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Also. I'm curious if maybe for better maintainability we should move the upstreamUnits logic into the systemd package proper.

@cleverca22 can you retarget at master now that 256 is out of staging?

@ElvishJerricco done!


Provide sample data with initial setup

Technology Provider organization with technical user for IaaS controllers in APP (user roles: technology manager and supplier)
OpenStack technical Service + Heat Template
Marketplace
Branding
....

The idea is:

On database initialization it should be possible to add this sample data if desired

Thus the technical tasks would be:

Create SQL files which provide only the sample data on top of a newly initialized databases
Enhance oscm-initdb so that if the database is initialized from scratch, additionally the SQL files are imported if a switch is set, otherwise the clean database is left as is

Before that we must define:

What exactly should the sample data be (which organizations, users, roles, services, marketplaces, etc.)?
How many sets of sample data will we offer? One set or several?

One set shall be fine for now. But it would be good to keep it exchangeable when enhancing oscm-initdb.


txt-bing-maps

Q
A

Bug fix?
No

New sample?
Yes

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Added sample code for column formatting a multi line of text field to show a Bing Map
Guidance
You can delete this section when you are submitting the pull request.

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'master' branch.

Thanks for your contribution! Sharing is caring.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: aprildunnam sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Bumped production RELEASE for 1.19-15

Bumped production RELEASE for 1.19-15
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/unhold


brew search throws undefined method error - macOS 10.13.6

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[x] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue? (Included below)

To help us debug your issue please explain:
What you were trying to do (and why)
I was attempting to run brew search <package name>. I am able to reproduce this issue regardless of the package specified.
What happened (include command output)
After running brew search, I received the following error message:
Error: undefined method `token' for nil:NilClass
Please report this bug:
  https://docs.brew.sh/Troubleshooting
/usr/local/Homebrew/Library/Homebrew/searchable.rb:28:in `block in search_string'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask.rb:20:in `block in each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/searchable.rb:27:in `select'
/usr/local/Homebrew/Library/Homebrew/searchable.rb:27:in `search_string'
/usr/local/Homebrew/Library/Homebrew/searchable.rb:7:in `search'
/usr/local/Homebrew/Library/Homebrew/extend/os/mac/search.rb:29:in `search_casks'
/usr/local/Homebrew/Library/Homebrew/cmd/search.rb:83:in `search'
/usr/local/Homebrew/Library/Homebrew/brew.rb:87:in `<main>'

What you expected to happen
Expected to receive a list of packages, from all taps, matching search string
Step-by-step reproduction instructions (by running brew commands)
$ brew search <package name>

Additional information
Output of brew doctor:
$ brew doctor
Your system is ready to brew.

Output of brew config:
HOMEBREW_VERSION: 1.7.1
ORIGIN: https://github.com/Homebrew/brew
HEAD: d6a245c05110c240e680da594c2cc6038998b82e
Last commit: 3 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 6517c2ef3174c76f1216e64d23923acf768e712c
Core tap last commit: 14 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_VISUAL: nano
CPU: octa-core 64-bit haswell
Homebrew Ruby: 2.3.7 => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Clang: 10.0 build 1000
Git: 2.17.0 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.54.0 => /usr/bin/curl
Java: N/A
macOS: 10.13.6-x86_64
CLT: <IP_ADDRESS>.1.1527767617
Xcode: N/A
XQuartz: 2.7.11 => /opt/X11

Can you post the output of brew tap?

Same here.
 ~/ brew doctor
Your system is ready to brew.

 ~/ brew config
HOMEBREW_VERSION: 1.7.1
ORIGIN: https://github.com/Homebrew/brew
HEAD: d6a245c05110c240e680da594c2cc6038998b82e
Last commit: 4 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 0e25f23d130a5e700786025f387a7507b80880e5
Core tap last commit: 7 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_NO_ANALYTICS_THIS_RUN: 1
CPU: quad-core 64-bit broadwell
Homebrew Ruby: 2.3.7 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.7/bin/ruby
Clang: 9.0 build 900
Git: 2.14.3 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.54.0 => /usr/bin/curl
Java: 1.8.0_181
macOS: 10.12.6-x86_64
CLT: <IP_ADDRESS>.1.1510905681
Xcode: 9.2
XQuartz: N/A

 ~/ brew tap
homebrew/cask
homebrew/cask-versions
homebrew/core
homebrew/fuse
kylef/formulae
mengbo/ch340g-ch34g-ch34x-mac-os-x-driver
pvinis/pvinis
universal-ctags/universal-ctags

@thenoelforte, the problem is spacemacs-nightly from pvinis/pvinis, which is empty.
cat /usr/local/Homebrew/Library/Taps/pvinis/homebrew-pvinis/Casks/spacemacs-nightly.rb
#cask 'spacemacs-nightly' do
  #version '2016-10-06_01-43-38-6e6c08815347a78008211613c9ddfa0acd7f0ef8'
  #sha256 '53a53cf2c62dcaada4ab6ec0e114ab9ca930688bbba455a90c6d3b316a7b0830'

 # url "https://emacsformacosx.com/emacs-builds/Emacs-#{version}-universal.dmg"
 # appcast 'https://emacsformacosx.com/atom/daily',
  #        checkpoint: 'e99da4f3923fdfcd3a28efae9fae7e624d070cc210bb45099ff570caef2eab00'
  #name 'Spacemacs'
  #homepage 'https://emacsformacosx.com/'
  #license :gpl

 # preflight do
    #system "mv", "#{staged_path}/Emacs.app", "#{staged_path}/Spacemacs.app"
    #system "rm", "#{staged_path}/Spacemacs.app/Contents/Resources/Emacs.icns"
   # system "curl -L \"https://github.com/nashamri/spacemacs-logo/blob/master/spacemacs.icns?raw=true\" -o #{staged_path}/Spacemacs.app/Contents/Resources/Emacs.icns"
  #end

 # app 'Spacemacs.app'
#end


Home Assistant MQTT documentation error

Describe the bug
https://github.com/danecreekphotography/node-deepstackai-trigger/wiki/Integrating-with-Home-Assistant
The documentation contains the following:
platform: "mqtt"
device_class: "connectivity"
name: "AI system status"
**state_topic: "node-deepstack-ai/status"**
payload_on: "online"
payload_off: "offline"
value_template: "{{value_json.state}}"

The state topic received is:
state_topic: "node-deepstackai-trigger/status"
Suggestion: for each camera MQTT binary sensor, I have added the following to get all JSON properties as attributes:
  json_attributes_topic: 'node-deepstackai-trigger/status'

Oops, thanks! I fixed it. Can you show me an example of your MQTT binary sensor?

i have a binary_sensor.yaml file
- platform: mqtt
  state_topic: "deepstack/driveway/motion"
  name: Camera Driveway motion AI
  device_class: motion
  payload_on: 'on'
  payload_off: 'off'
  value_template: '{{value_json.state}}'
  json_attributes_topic: 'deepstack/driveway/motion'

Don't have an example of the output of the camera properties.
Here are is my AI status
- platform: mqtt
  device_class: connectivity
  name: 'Deepstack AI status'
  state_topic: 'node-deepstackai-trigger/status'
  payload_on: 'online'
  payload_off: 'offline'
  value_template: '{{value_json.state}}'
  json_attributes_topic: 'node-deepstackai-trigger/status'

Perfect, thank you. Great idea! Documentation's updated.


Add possibility to ignore some ids

Enable user to ignore messages sent by some ids. Messages with one of these ids will not be displayed.
Ids can be written in base 2, 8, 10 and 16 as long as they are correctly prefixed ('0x', '0o'...)
They can be provided directly on the command line or in a text file.

Thanks again for your contributions. I had to solve a merge conflict on this one, hope I didn't break anything! Unfortunately I don't have any CAN device where I am to test it :)


1239 visualize decision trees

Fixes #1239

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Parthiv Naresh seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@ParthivNaresh I'm excited to review this! Could you please include an example of the output in the PR description? Will make it quick for reviewers to understand what's up.


Timeline: Text inside bars (feature-request)

Is it possible to show text inside the timeline-bars?
Usecase:
Each line/sections is an employee and shows their active time working on different orders. We want to show the order name or number inside the bars.

This would be a great feature to have, but as of now it's not supported.


Table body doesn't resize on fullscreen setup

I'm submitting a ...
[ x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
When using a fullscreen table inside an element with another component that has a dynamic/changeable height the table doesn't resize until the window is resized.
Expected behavior
The table body will dynamically grow to its full height
Reproduction of the problem
http://plnkr.co/edit/brWKmnNmjRxRPcvkSgrZ?p=info
Resize the Preview window so that you can see the bottom scrollbars. Then hover over the element containing "top". The table body will then be pushed down while the surrounding div and the actual ngx-datatable element will resize properly.
What is the motivation / use case for changing the behavior?
When used inside a flex layout with another dynamic element like e.g. a table dynamic filter where search fields can be added dynamically you'd always have to manually call window.dispatchEvent(new Event('resize')); to manually trigger a resize.
Please tell us about your environment:
Windows 7, IntelliJ, npm

Table version: 9.1.0

Angular version: 4.1.3

Browser: Tested on Chrome 58.0.3029.110 and Firefox 53.0.2 though probably affects all Browsers

Language: Typescript 2.3.2 with es2016 setup

With a little bit of css "hacking" this would be a possible workaround though having it natively supported would be better:
ngx-datatable{
   position: absolute !important;
   height: auto !important;
   top: 0;
   left: 0;
   right: 0;
   bottom: 0;
   width: 100%;
   >div.visible {
     height: 100%;
     display: flex;
     flex-direction: column;
     >datatable-body{
       flex-basis: 100%;
     }
   }
 }

I wouldn't really consider this a hack, its common CSS practices.

@ExTheSea baseado na sua solução:
.ngx-datatable{
position: absolute !important;
height: auto !important;
top: 0;
left: 0;
right: 0;
bottom: 0;
width: 100%;
}
.ngx-datatable.div.visible {
height: 100%;
display: flex;
flex-direction: column;
}
.ngx-datatable.datatable-body{
flex-basis: 100%;
}


Corrected typo in the note

Changed 'us' to intended 'use'

@shahid21st Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@shahid21st Thank you for signing the Contributor License Agreement!

Codecov Report

Merging #625 into master will not impact coverage.

@@          Coverage Diff           @@
##           master    #625   +/-   ##
======================================
  Coverage    72.1%   72.1%
======================================
  Files          70      70
  Lines        2305    2305
  Branches      380     321   -59
======================================
  Hits         1662    1662
  Misses        493     493
  Partials      150     150

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 937027b...db39a5f. Read the comment docs.


Update install-zos.md

Troubleshooting for missing auto conversion. https://www.ibm.com/support/knowledgecenter/en/SSLTBW_2.3.0/com.ibm.zos.v2r3.bpxb200/setupa.htm

Hello @JakubSkrivanek and @John-A-Davies!
I am looking into the zowe-check-prereqs.sh script written by @John-A-Davies . This script is intended to be executed before the installation. It checks the value of _BPXK_AUTOCVT and if it is set then if issues a warning.
My understanding is that Zowe installation expects the auto conversion to be off so all the files are created in the same encoding EBCDIC. Then the Zowe started tasks does not need _BPXK_AUTOCVT=ON. _BPXK_AUTOCVT=ON will help with the files that were stored in ASCII and tagged during installation but it can affect other files. Since the Zowe has been tested with without _BPXK_AUTOCVT then I would recommend to install Zowe without it.
@stevenhorsman, do you know if we support _BPXK_AUTOCVT=ON during installation or we expect to have it off?
@JakubSkrivanek, what has been the value of _BPXK_AUTOCVT during your installation? One way how to find out is to run zowe-check-prereqs.sh or issue echo $_BPXK_AUTOCVT in your z/OS UNIX environment.

Hi, I have initially reported the issue for version 1.0.0 NB. My result of running the prereq scrip was:
Check the USS AUTOCVT
OK: OMVS AUTOCVT is set to OFF
my  echo $_BPXK_AUTOCVT command showed value ON
I have moved to version 1.0.0 RC2 since. Running the prereq scrip now, I can see:
Check USS AUTOCVT
OK: OMVS AUTOCVT is set to OFF
Warning: _BPXK_AUTOCVT is set to ON
OK: _BPXK_CCSIDS is not set
I have changed the setting of  _BPXK_AUTOCVT  to stay on OFF which resulted in problems during run of the zowe install script.
FSUM7332 syntax error: got (, expecting Newline to be exact.
Looking at my environment set
, I have path set including Rocket/python36.
There is SED available in that path which messes up the coding while running the installation script.
Commenting out all the paths to this folder and the BPXK, _CEE_RUNOPTS, and TAGs resulted into a correct run of the installation script.

Thanks.  Please indicate which script produced the error FSUM7332 syntax error: got (, expecting Newline.  Please show me what that file looks like in ISPF 3.17 when you VIEW it and also when you type MF against its name.
The warnings from verify scripts about AUTOCVT are merely advisory - we don't want to change the user's environment permanently and we don't want to mandate or deprecate such settings.

FSUM7332 syntax error: got (, expecting Newline to be exact.
Looking at my environment set
, I have path set including Rocket/python36.
There is SED available in that path which messes up the coding while running the installation script.
Commenting out all the paths to this folder and the BPXK, _CEE_RUNOPTS, and TAGs resulted into a correct run of the installation script.

I have reported the problems with sed in this item - https://github.com/zowe/zowe-install-packaging/issues/381. The workaround was to update the Jakub's profile and remove Rocket's Python directory from the PATH.

@John-A-Davies @plavjanik Hi John, Petr,
Reading the discussion thread, it seems there is already a solution to the problem that Jakub encountered. As a follow-up, is the doc changes in this PR valid at this point? If not, do you think we could draft a troubleshooting topic to help people who might meet the same issue as Jakub? Thanks!

FYI, if you plan to include this with Zowe 1.6.0 release you can merge to doc-staging now and the content will be published at the proper time.


New flag to scrub IP addresses

Implement IP address scrub.

MacOS CI tests failure can be ignored for now.

@maxgolov Do you have plan to mark it for review, or else close if this is not valid ?

@lalitb - we need it, but we identified an extra scenario that's not covered by the patch. I'll do another iteration on it.


Add Control Debug Telemetry Data Pipeline

This is the software side of the initial prototype to get motion control debug data into the ROS ecosystem in real time for logging and analysis purposes.

This obviously needs testing on hardware, then we should have a discussion about meaningful things like code quality and then probably a discussion about names. I know the software-communications packets have different names for stuff than the ROS messages. Software and firmware motion nomenclature was already conflicted before these additions, so perhaps we should merge this as is and then do a big cleanup of packet structure and names.
No action needed at this time.

added wheel vels, and imu. Needs testing. Especially look at wheel vels, firmware and software do not use the same ordering for wheels in float[4] arrays representing wheel state


Infer route parameter type from matcher's guard check if applicable

closes #8137 - Just not for $page.params
Currently all route parameters always have type string. This type is correct, but often broader than it needs to be.
If a matcher is used on a route parameter, and that matcher is annotated with a guard check, we can infer the type much more narrowly.
// src/params/ab.ts
export const match = (param) : param is "a" | "b" => param == "a" || param == "b";

// src/routes/[myParam=ab]/+page.ts
export async function load({params}) {
    params.myParam //Has type "a" | "b"
}

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Thanks for the review @dummdidumm ! I'm going to merge this based on your approval


Tailwindcss conflicts with new env handling

Describe the bug
I'm using tailwind and have the following app.css:
@tailwind base;
@tailwind components;
@tailwind utilities;

@layer base {
  @font-face {
    font-family: 'Oswald';
    src: url(/fonts/oswald/Oswald-VariableFont_wght.woff2) format('woff2'),
      url(/fonts/oswald/Oswald-VariableFont_wght.ttf) format('truetype');
    font-display: swap;
  }
}

Now all layouts that import the css file throw this error:
Error: Cannot import $env/static/private.js into client-side code:
-<EMAIL_ADDRESS>  - src/app.css
    - src/init/index.ts
      - src/lib/import/afterbuy-import-jobs.ts
        - src/lib/import/afterbuy-catalogs/pipeline.ts
          - src/lib/import/afterbuy-catalogs/stream-from-afterbuy.ts
            - src/lib/afterbuy.ts
              - $env/static/private.js

When I remove the tailwind directives or the whole import everything works but the styles are broken of course.
Reproduction
Just add the default tailwind css directives.
Logs
No response
System Info
  System:
    OS: Linux 5.15 NixOS 22.11 (Raccoon) 22.11 (Raccoon)
    CPU: (8) x64 Intel(R) Core(TM) i7-7700T CPU @ 2.90GHz
    Memory: 16.14 GB / 31.28 GB
    Container: Yes
    Shell: 5.1.16 - /run/current-system/sw/bin/bash
  Binaries:
    Node: 14.20.0 - /nix/store/fg072dha8d8nfhzcsxxr1nybx64pxgp1-nodejs-14.20.0/bin/node
    npm: 6.14.17 - /nix/store/fg072dha8d8nfhzcsxxr1nybx64pxgp1-nodejs-14.20.0/bin/npm
  Browsers:
    Chromium: 103.0.5060.114
    Firefox: 102.0.1
  npmPackages:
    @sveltejs/adapter-node: 1.0.0-next.85 => 1.0.0-next.85
    @sveltejs/kit: 1.0.0-next.395 => 1.0.0-next.395
    svelte: ~3.49.0 => 3.49.0
    vite: ~3.0.0 => 3.0.0

Severity
blocking an upgrade
Additional Information
No response

From the bug template:

A link to a repository, or a fork of https://node.new/sveltekit, that reproduces the issue. Reproductions must be short, self-contained and correct and must not contain files or code that aren't relevant to the issue — please do NOT just paste a link to your project. Explaining how to reproduce is generally not enough. It pushes the burden of creating a reproduction project onto a small set of volunteer maintainers and isn't scalable. If no reproduction is provided, the issue will be closed.

I tried:

Just add the default tailwind css directives.

Everything worked just fine.  We'll close this if there isn't a reproduction soon

Damn I cannot reproduce this either in a clean repo...
But why on earth does app.css depend on anything at all?

Can you share the whole repo with us? Perhaps privately? I believe there's an issue here, but it seems harder to cause it

I will try to when I get around to it. The project is part of a larger rush.js monorepo so there is much going on. I will try to slim it down as much as possible.
btw: I planned migrating from tailwind to windi and used this opportunity to finally bite the bullet and the problem did not surface again.

i have exactly the same issue on a sveltekit/tailwind project since the new env handling. it builds again as soon the
@tailwind base; @tailwind components; @tailwind utilities; @tailwind variants;
lines are removed from my css file.

@benjaminlewandowski
We have not been able to reproduce this issue. Can you file a new bug with the bug template and a reproduction? I'm sure there's a bug here -- we just can't make it happen.


GDS Plugins: Start-Up Applications

Related Issue(s)

Has Unit Tests (y/n)

Documentation Included (y/n)

Change Description
This PR makes several changes:

Adds a GdsFunction plugin with generic run function
Adds a GdsApp plugin with get_process_invocation function
Fixes CLI for argument parsing to add in plugin sections for arguments
Adds UTs for testing both plugin types
Straightens out naming for plugins
Adds Plugin class wrapper helper

Rationale
Users want start-up applications and functions
Future Work
Document this in fprime (github.com/nasa/fprime)

Also note: the plugin function definitions were moved within the classes they should return.


Polling time is not configurable

We are facing throttling on describeStacks api. Is it possible to add a configuration to configure the polling interval?
Following code looks hardcoded 5000 ms:
https://github.com/serverless-stack/sst/blob/a8f56ae888d3860e126ed88ea539389985b5da9e/packages/cli/scripts/remove.mjs#L109
https://github.com/serverless-stack/sst/blob/a8f56ae888d3860e126ed88ea539389985b5da9e/packages/cli/scripts/util/cdkHelpers.mjs#L365

created PR here - https://github.com/serverless-stack/sst/pull/1872 for v0

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


tracelib.c: Removed unnecessary condition in if

if ((kev.flags & EV_ERROR) == 0 || ((kev.flags & EV_ERROR) > 0 && kev.data != 0))
If compilers find first operand true while evaluating || , the second operand isn’t checked.
So the second operand is only checked if (kev.flags & EV_ERROR) != 0.
So (kev.flags & EV_ERROR) > 0  it doesn’t need to be checked in conjunction with kev.data != 0.
It doesn’t return negative.(tested)

Please rebase your branch. https://trac.macports.org/wiki/WorkingWithGit#upstream-fetch

It doesn’t return negative.(tested)

To determine this by testing, you would have to generate every possible error condition and see what is returned. I really doubt you did that. :)
Fortunately, you don't actually have to test because flags is a uint16_t and thus can't be negative by definition.

One more thing: this pattern is repeated in a few other places in the file. Why not change all of them?

On Mon, 1 Apr 2019, 07:35 Joshua Root<EMAIL_ADDRESS>wrote:

It doesn’t return negative.(tested)
To determine this by testing, you would have to generate every possible
error condition and see what is returned. I really doubt you did that. :)
Fortunately, you don't actually have to test because flags is a uint16_t
and thus can't be negative by definition.

Glad u told me that uint16_t. ^_^
i tried to research there types but cudnt really find.
By tested, i didn't mean like that.
There is already no handling for negative case even if it arises.
Also there are only three flags being formed in there EV_ADD, EV_RECEIPT &
EV_ERROR. I just researched on them. and pretty much combination of these 3
just give arise to 8 conditions.
That is a what i meant to say.
Sorry if i was wrong, i will take more care from next time.
Regards,
Mihir
—

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/macports/macports-base/pull/119#issuecomment-478411282,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ao9le63lnqQi840im14NzBssoy0lyVleks5vcWl5gaJpZM4cUVRv
.

On Mon, 1 Apr 2019, 07:38 Joshua Root<EMAIL_ADDRESS>wrote:

One more thing: this pattern is repeated in a few other places in the
file. Why not change all of them?
To be honest, i noticed that now.
I will rebase my branch and change it if it makes sense. :)

Regards
Mihir

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/macports/macports-base/pull/119#issuecomment-478411663,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ao9le9O03rShIJvO6IoZL3-rKnXrOjC4ks5vcWodgaJpZM4cUVRv
.

Glad u told me that uint16_t. ^_^
i tried to research there types but cudnt really find.

If you look at the kevent man page, it refers to a few headers, one of which is sys/event.h. That contains the definition of struct kevent.

Aah, srry for that. My mistake! That didn't come to my attention.
Will be more careful next time.

No need to apologise. This is meant to be a learning experience after all. :)

No need to apologise. This is meant to be a learning experience after all. :)

Thanks ^_^,
I will rebase my branch and submit a new PR :)

Oh, there should be no need for a new PR. If you update your fork with the commits from macports/macports-base, the PR will update automatically.
One note: the usual GitHub procedure would be to create a new branch in your fork for each PR, and then delete the branch after the PR is merged. That way you can keep master in your fork the same as master in our repo.

Thanks for the helpful info, I will do exactly that. :)
Mihir


get Request Fail when load quiz

error message is:

Because Leetcode website has changed their website content. We can't get the content in old way. I'm trying to figure out a new way. Thanks

I should say thanks, great work, this project is very helpful.

Thanks, that means a lot to me!

Fixed


[Bug] Exception in Get-DbaRegServer.ps1 caused by incorrect path to APPDATA

Environmental information
#### PowerShell version:

Name                           Value
----                           -----
PSVersion                      5.1.18362.145
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0, 5.0, 5.1.18362.145}
BuildVersion                   10.0.18362.145
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

#### dbatools Module version:
Name    : dbatools
Path    : C:\Program Files\WindowsPowerShell\Modules\dbatools\1.0.19\dbatools.psd1
Version : 1.0.19

Report
The $Home variable used on line 118 does not necessarily lead to the APPDATA location in certain profiles using folder redirection.
        if (-not $PSBoundParameters.SqlInstance -and -not ($IsLinux -or $IsMacOs)) {
            $null = Get-ChildItem -Recurse "$home\AppData\Roaming\Microsoft\*sql*" -Filter RegSrvr.xml | Sort-Object LastWriteTime -Descending | Select-Object -First 1
        }

perhaps should be
        if (-not $PSBoundParameters.SqlInstance -and -not ($IsLinux -or $IsMacOs)) {
            $null = Get-ChildItem -Recurse "$env:APPDATA\Microsoft\*sql*" -Filter RegSrvr.xml | Sort-Object LastWriteTime -Descending | Select-Object -First 1
        }

Host used

[x ] powershell.exe
[ ] ISE
[ ] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[ ] Still buggy in powershell.exe

Errors Received

writeErrorStream      : True
PSMessageDetails      :
Exception             : System.Management.Automation.ItemNotFoundException: Cannot find path 'H:\AppData\Roaming\Microsoft\' because it
                        does not exist.
                           at System.Management.Automation.LocationGlobber.ExpandMshGlobPath(String path, Boolean allowNonexistingPaths,
                        PSDriveInfo drive, ContainerCmdletProvider provider, CmdletProviderContext context)
                           at System.Management.Automation.LocationGlobber.ResolveDriveQualifiedPath(String path, CmdletProviderContext
                        context, Boolean allowNonexistingPaths, CmdletProvider& providerInstance)
                           at System.Management.Automation.LocationGlobber.GetGlobbedMonadPathsFromMonadPath(String path, Boolean
                        allowNonexistingPaths, CmdletProviderContext context, CmdletProvider& providerInstance)
                           at System.Management.Automation.LocationGlobber.GetGlobbedProviderPathsFromMonadPath(String path, Boolean
                        allowNonexistingPaths, CmdletProviderContext context, ProviderInfo& provider, CmdletProvider& providerInstance)
                           at System.Management.Automation.SessionStateInternal.GetChildItems(String path, Boolean recurse, UInt32 depth,
                        CmdletProviderContext context)
                           at Microsoft.PowerShell.Commands.GetChildItemCommand.ProcessRecord()
TargetObject          : H:\AppData\Roaming\Microsoft\
CategoryInfo          : ObjectNotFound: (H:\AppData\Roaming\Microsoft\:String) [Get-ChildItem], ItemNotFoundException
FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.GetChildItemCommand
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at Get-DbaRegServer<Process>, C:\Program Files\WindowsPowerShell\Modules\dbatools\1.0.19\allcommands.ps1: line
                        31429
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {0, 1, 0, 0}

Steps to Reproduce
Run Get-DbaRegServer without parameters for a user whose APPDATA folder doesn't reside in $HOME.
Expected Behavior
Use a more reliable path to APPDATA. Otherwise, trap the exception.
Actual Behavior
Exception not trapped.

thanks so much, Jared! Any chance you can fix this one too? 😅 I'm knee deep in a work project.

No problem.
http://jaredfaulkner.net?promo=email_sig&utm_source=product&utm_medium=email_sig&utm_campaign=edit_panel&utm_content=thumb
Jared Faulkner
https://jaredfaulkner.net
https://jaredfaulkner.net?promo=email_sig&utm_source=product&utm_medium=email_sig&utm_campaign=edit_panel&utm_content=thumb
On Thu, Jul 11, 2019 at 1:14 AM Chrissy LeMaire<EMAIL_ADDRESS>wrote:

thanks so much, Jared! Any chance you can fix this one too? 😅 I'm knee
deep in a work project.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/issues/5891?email_source=notifications&email_token=AMR5NYBOPM56PU3KMDPXK3TP63FUTA5CNFSM4H7TFX62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVT5GA#issuecomment-510344856,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMR5NYGTIX2BH2VQSUKMWG3P63FUTANCNFSM4H7TFX6Q
.

you're awesome! 💪 Thank you so very much


My Insiders can not get updated

I have VS Code Insiders from yesterday on macOS. c1bca6d7cc2c5f71ae04deda90c0cf50819ffde4
There is a newer VS Code Insiders released
I click on Check for Updates in VS Code Insiders. Nothing happens

Where do we keep logs so I provide more details?
I can repro if I restart VS Code Insiders.
Yesterday I successfully updated.

Issue was in my network for some reason. Once I changed networks all worked fine. Closing this.


Problem in lenses: show all tab fields on index table

Hi! Example lense returns all fields from resuorce:

class ResourceLens extends Lens {

    use HasResourceNavigationTabTrait;

    public function fields(Request $request) {
        $res = new Resource(new \Models\Model() );

        // return all the same fields
        return $res->fields($request);
    }

}

Resource fileds grouped in ResourceNavigationTab. But in Lens index table displayed all fields with fields from ResourceNavigationTab.

Workaround:
        return collect($res->fields($request))->filter(function ($value, $key) {
            return false === $value instanceof ResourceNavigationTab;
        })->toArray();

Well, this package was only meant to be used on the details view, not on index. so I will discard this as a bug on index view you can use this other package: https://github.com/dcasia/resource-navigation-link which only works on the index and looks exactly the same as this one


[BEAM-13925] PR-Bot - Dont double assign committers if author or other reviewer is a committer

Right now, the pr-bot automatically assigns a committer after the first PR review, even if the initial reviewer is a committer. This updates to avoid doing that.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

R: @lostluck


Accessibility Audit - Carbon Website: Components - File uploader

This epic is tied to a larger "release" that is meant to track the progress of the Accessibility Audit for the Carbon Website. We will audit all of the pages of the carbon website based on AVT 1, AVT 2, & AVT3 - which can be found here.

AVT 1: will audit the site against the IBM Accessibility Checker - browser extension

AVT 2: will audit the site against accessible keyboard interactions

AVT 3: will audit the site against screen readers

Legend:

🌶 fully audited - on all three AVT levels

:one: 🔍 AVT 1 audited
:two: 🔍 AVT 2 audited
:three: 🔍 AVT 3 audited

:boom: issues and documentation - on all three AVT levels are complete

:one: 🚨 AVT 1 issues submitted
:two: 🚨 AVT 2 issues submitted
:three: 🚨 AVT 3 issues submitted

:sparkles: tickets are triaged

Current status of this page: 1⃣ 🔍 AVT 1 audited & 🚨 AVT 1 issues submitted

Components - File uploader

Usage: 8 issues - no new issues
Style: 3 issues - no new issues
Code: 8 issues - no new issues
Accessibility: 3 issues - no new issues

Closing due to inactivity. Will reopen if/when ready to be worked on again.


Not work with RN 0.56.0

A problem occurred configuring project ':app'.

Could not resolve all dependencies for configuration ':app:_debugApk'.
A problem occurred configuring project ':react-native-smart-splashscreen'.
The SDK Build Tools revision (23.0.3) is too low for project ':react-native-smart-splashscreen'. Minimum required is 25.0.0

PR https://github.com/react-native-component/react-native-smart-splash-screen/pull/35 fixes this issue, but like many other PRs they haven't been merged. You can either fork it or check other forks that have updated versions.


Feature: tslint install <config> command

I wonder if it makes sense to create this tslint install <config> command.
I write and use my own config: https://github.com/unional/tslint-config-unional
and one of the main objectives of my config is to define a coding style that works across IDEs.
I'm able to do that by finding common possible settings so that the IDE can be configured in a way that works seamlessly with the style.
However, that also means that the user needs to configure their IDE of choice to a specific setting out-of-band.
This is ok in company environment but for open source project, it is crucial to set up the IDE setting with minimal friction.
If we introduce tslint install <config>, we can add a function in the config that can create these settings in the project automatically.
Currently, my "workaround" would involve a custom cli tool that will scaffold the project with these settings to be checked in into github.
Oh, by the way, it would be even better if we can write some tools that will read the tslint config file and output config settings for different IDEs. 🌷

You're probably trying to solve your problem on the wrong end.
IMO tslint should not be responsible to configure any IDE based on your tslint.json.
Instead you could write your own tool that installs tslint with your config and has plugins to configure your preferred IDEs. Maybe that's what you meant and already implemented as "workaround".

Yes, I would agree that is outside the responsibility of the linting service.
I'm looking at this on the consuming side and it doesn't help if the IDE you use doesn't collaborate with it.
If you consider tslint has two components: linting service and cli, this may be suitable in the cli component as it can take the responsibility of configuring the environment (and in a sense similar to ESLint providing the eslint:recommend rule set).
Currently, it is extra work from the user to get things setup and ready.
For contributing to open source project, many times you would fork a repo, make some fix, and create a PR.
You do not need to work on it in the future for the most part, and this (or similar solution) would help making that experience more streamlined.

Seems like consensus is that this should be an external tool. Closing as external, but please do post here if you think this should be a part of TSLint itself!


Question: Is this project still maintained?

Hey, I'm looking for a javascript bar code scanner library, and I stumbled over this project, but from what I can see its last release was way back in 2017.
So my  question is: should I pick this library or shall I look for another one, which is actively maintained?
Thanks

Until Serratus decides to return, or designates some other official continuation, I maintain a fork at https://github.com/ericblade/quagga2

@ericblade  Thanks for letting me know! I will switch to quagga2!

Maybe you should put a note on the quagga README file to redirect users to quagga2.
For some reasons,  Google didn't provided any references to quagga2, when searching for barcode reader javascript libs

I don't want to make it appear that Serratus officially supports my fork as a continuation, unless he voices that opinion.

Just create a pull request and assign it to him.  You will find out that way his opinion :)

@serratus what's your opinion regarding my request above?


Database Buttons

Laravel Version: 5.4
Voyager Version: 0.11.6
PHP Version: 7
Database Driver & Version:

Description:
No shortcut keys on database

Add Timestamp Fields

Add Soft Delete Field

@marktopper

Duplicate of #921

Right now, you can manually add created_at, updated_at, deleted_at columns and set their type to TIMESTAMP.


Fix shutdown on Ctrl+C for Python source stages

Description

Ensure that Python sources which run indefinitely actually shutdown when requested by the pipline (ex: when the user hits Ctrl+C).
The KafkaSourceStage already handled this properly, move the stop method from this class into the SingleOutputSource.
Add request_stop and is_stop_requested methods to SingleOutputSource.
Update all existing source stages which run indefinitely.
Add new should_stop_fn constructor argument to DirectoryWatcher, sources which use the watcher (AutoencoderSourceStage & AppShieldSourceStage) pass in their is_stop_requested method allowing the watcher to know when to shut down.
Move the RSS source generator method from the RSS source module into the RSSController, and update the RSSSourceStage to use the RSSController directly rather than the RSS module. This avoids the problem where modules require all config values to be JSON serializable, preventing a callback function from being passed to a module.

Closes #1837
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

/merge


No shortcuts displayed for plugins

Now that #3300 reenables correct plugin loading, I notice that shortcuts are not displayed in the menus for some plugins. For me the shortcut is shown for breakpoints, but not for pylint or profiler. The 3 of them have configurable shorcuts, and I noticed no difference between the declarations.
For my other external plugins which use fixed shortcuts, the shortcuts are not displayed either.
BTW API-wise, in both cases it feels strange to define the actions separately from the shortcuts.

Hi,
Found your post when I could not find the profiler in Spyder after upgrading to 3.0.0b4. Is there any way, right now, that I can still use the shortcuts that you mention?

@NGJROMO, the profiler was not packaged as part of beta4. This was our mistake, and it was already fixed in #3300 :-)

Right, thanks for the quick answer. I'll look forward to it coming back in beta5 then. Any ETA for that or do you recommend downgrading to beta3 if I need the profiler?

@Nodd, I'm not seeing this problem on Linux. Could you post your installation details?

I also don't see a problem with pylint or profiler. For external plugins with fixed shortcuts: they are displayed if the shortcut is configured in utils.qthelpers.create_action(), as done here, but I can't get it to work using config.gui.fixed_shortcut(), see e.g. here. In fact, the first method seems cleaner anyway.


[CT-2458] [Bug] Package-level model default config taking precedence over model-specific config in another package

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
In project B, we include project A (which includes models) as a package:
# b/packages.yml
packages:
  - local: ../a

Models in project A have specific config() within the model SQL file:
-- a/models/some_incremental_model.sql
{{ config(materialized="incremental", incremental_strategy="alpha") }}
...

While project B sets a default value for some of the same keys in dbt_project.yml:
# b/dbt_project.yml
models:
  +incremental_strategy: beta

When we compile project B, models in project B behave as expected - they use a incremental_strategy configured in the sql file if set, or the default ("beta") if not. However, models from project A all have their incremental_strategy overwritten with "beta", even if they had a more specific configuration in their SQL file.
This seems surprising, since I expected the more specific config in Project A to "win".
(This is not specific to incremental_strategy, and the same effect can be found with other config keys).
Expected Behavior
Per https://docs.getdbt.com/reference/model-configs#configuring-models, I expect the most specific configuration - within the SQL file in project A - to win, rather than the default package configuration in project B.
Steps To Reproduce
As per the Current behavior above, this appears to require:

Two projects, one referenced as a package by the other.
SQL files in the first project contain some model-specific config
The second project sets a default for the same config key
dbt compile shows that the second project's default config wins over the first project's seemingly more specific config.

Relevant log output
No response
Environment
- OS: Ubuntu 20.04
- Python: 3.10.6
- dbt: 1.4.6

Which database adapter are you using with dbt?
redshift
Additional Context
Possibly related to https://github.com/dbt-labs/dbt-core/issues/6705

@chronitis Sorry that this surprised you. This is documented in https://docs.getdbt.com/docs/build/packages:

Configurations made in your dbt_project.yml file will override any configurations in a package (either in the dbt_project.yml file of the package, or in config blocks).

It sounds like we need to update the docs in https://docs.getdbt.com/reference/model-configs (and https://docs.getdbt.com/reference/configs-and-properties) to reflect this fact as well!
The intent of this behavior is to enable users of packages that they don't own/maintain to make changes to models from those packages, without needing to copy-paste-edit the code themselves.
I'm going to close this issue, and open a docs one instead.

@jtcohen6 thanks for the quick response and explaining the design decision here.
I had read that page, but I'd clearly not taken in that part. I can see the argument that it makes sense to be able to reconfigure imported models, although I did still find the apparent inversion in precedence surprising.


Auth for Sandbox

How do I set the instance to sandbox?  Currently
UsernamePasswordAuthenticationFlow(clientId, clientSecret, username, password) set to  prod by default.

You need to point to sandbox end-point:  https://test.salesforce.com/services/oauth2/token
More info at the wiki wiki


merge from template-node-app

.gitignore

chore: add /node_modules/** ()
chore: remove ./node_modules/** ()

README.md

docs(README): update from template

:tada: This PR is included in version 2.0.133 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[SPARK-48773][FOLLOW-UP] spark.conf.set should not fail when setting spark.default.parallelism

What changes were proposed in this pull request?

spark.session.set should not fail when setting spark.default.parallelism.
Why are the changes needed?

This is to fix a behavior change where before SPARK-48773, set spark.default.parallelism through spark session does not fail and is a no op.
Does this PR introduce any user-facing change?

Yes.
before SPARK-48773, spark.conf.set("spark.default.parallelism") does not fail and is a no-op.
after ``SPARK-48773, spark.conf.set("spark.default.parallelism") will fail with a CANNOT_MODIFY_CONFIG` exception.
With this followup, we restore the behavior to spark.conf.set("spark.default.parallelism") does not fail and is a no-op.
How was this patch tested?

manually testing.
Was this patch authored or co-authored using generative AI tooling?

No

The streaming test failure is unrelated, I'm merging it to master, thanks!


Allow using codespan without codespan-reporting

This copies the implementation of line_starts to codespan. It also adds --no-default-features to CI.
r? @brendanzab

Awesome!


Update/clap + feature must provide one arg list + fan exclusive

Hello,
While testing I was surprise by just typing jbod list and getting nothing, I was a little bit puzzled, did this to help a little bit with command discoverability.
Why updating clap? arg_required_else_help was not present in 2.31.2 version.
If my reading of enclosure_overview function is correct: at least any of the arguments or multiple must be provided and some are exclusive  to subcomand jbod list no arguments do nothing.
Here what it does in case of no arguments were passed with these change:
./target/debug/jbod list
jbod-list
list

USAGE:
    jbod list [OPTIONS]

OPTIONS:
    -d, --disks        List disks
    -e, --enclosure    List enclosure
    -f, --fan          List fan
    -h, --help         Print help information

Also added a way to say --fan is exclusive to other arguments. Before it was possible to do -df and fan info were not printed.
./target/debug/jbod list -fe
error: The argument '--fan' cannot be used with one or more of the other specified arguments

USAGE:
    jbod list [OPTIONS]

For more information try --help

EDIT: edited to remove word duplication and typo + rephrasing first paragraph.

Tested it on my dev-computer, since this change-set doesn't touch feature it shall be fine

Thanks for the patch!


Divider docs incorrectly state a Divider is one logical pixel in height.

http://docs.flutter.io/flutter/material/Divider-class.html

Why is that incorrect?


memory leak

var myvideo=req.files.myvideo[0].originalname;
var ext = path.extname(''+myvideo+'');
const gcsname =Date.now()+ext;
console.log('Hello wold ');
console.log(gcsname);
	  const file = bucket.file(gcsname);

	  const stream = file.createWriteStream({
	    metadata: {
	      contentType: req.files.myvideo[0].mimetype
	    },
	    resumable: false
	  });

	  stream.on('error', (err) => {
	    req.files.myvideo[0].cloudStorageError = err;
	    req.videocheck=false;
	    console.log('Error function hit.....');
	      mystreamendfun();
	    next(err);
	  });

	  stream.on('finish', () => {
	    req.files.myvideo.cloudStorageObjectimg = gcsname;
	    file.makePublic().then(() => {
	      req.files.myvideo[0].cloudStoragePublicUrlimg =getPublicUrl(gcsname);
	      req.videocheck=true;
         console.log('Finish function hit.....');
	      mystreamendfun();

	      next();
	    });
	  });

 stream.end(req.files.myvideo[0].buffer);

Hi @jaman99 , thank you for the detailed bug report. Unfortunately, this issue tracker has limited visibility and scope - basically, it's just for reporting issues with the samples contained here and is only watched by the folks who work on these samples.
We are happy to help you move this bug to venues where it'll get the attention of the appropriate product teams.

StackOverflow's Google Cloud storage tag is most appropriate for design and general troubleshooting. You can read more about community support here.
Google Cloud's public issue tracker is most appropriate for bugs and unexpected behavior.
Google Cloud Support is most appropriate if you have an existing support contract with Google.

I'm going to close this issue because the folks who own this repo have no actionable way to resolve it directly, but please let us know if there's anything we can do to help!


Missing options to cygwin bash

Just tried Terminus 1.0.0 alpha 19 on Windows 10 with Cygwin bash.
What I've noticed is that the PATH is not properly set (e.g. /usr/bin/ is not included in the PATH).
Comparing to the cygwin.bat terminus just run bash, while the bat file runs bash --login -i

As a workaround you may create a batch file alongside bash.exe:
@ECHO OFF

"%~dp0bash.exe" --login -i

You can modify config.yaml in %APPDATA%\terminus to use the batch instead of the exe.


Libreoffice langpack

Since #19 by MichalMMac in  sebtomasi-recipes was not merged within one year (and the last commit to sebtomasi's repo is more than two years ago), we decided to host a newer version in our repo.
Additional features are:

Inclusion of glob to allow downloads again (of course)
Dynamic determination of minimum_os_version

To be done:

Create a PR for the download recipe of @hjuutilainen (at this time, only x86_64 is supported, because the parent download is hard-coded to that)

We decided not to include the jss recipe, since we do not use the JAMF importer and cannot check PR's against it.

Got an error running this recipe:
The following recipes failed:
    ./LibreOffice/LibreOfficeLangPack.munki.recipe
        Error in com.github.wycomco.munki.LibreOfficeLangPack: Processor: PlistReader: Error: Path '/Users/choules/Library/AutoPkg/Cache/com.github.wycomco.munki.LibreOfficeLangPack/app_libreoffice_still_de/Applications/LibreOffice.app/Contents/Info.plist' doesn't exist!

No time found to dig deeper, for today.

Got an error running this recipe:
The following recipes failed:
    ./LibreOffice/LibreOfficeLangPack.munki.recipe
        Error in com.github.wycomco.munki.LibreOfficeLangPack: Processor: PlistReader: Error: Path '/Users/choules/Library/AutoPkg/Cache/com.github.wycomco.munki.LibreOfficeLangPack/app_libreoffice_still_de/Applications/LibreOffice.app/Contents/Info.plist' doesn't exist!

No time found to dig deeper, for today.

Should work now. Tests both vanilla and with override ran fine.


Fix/getty eof

What this PR does:
Which issue(s) this PR fixes:
Fixes #
You should pay attention to items below to ensure your pr passes our ci test
We do not merge pr with ci tests failed

[ ] All ut passed (run 'go test ./...' in project root)
[ ] After go-fmt ed , run 'go fmt project' using goland.
[ ] Golangci-lint passed, run 'sudo golangci-lint run' in project root.
[ ] After import formatted, (using imports-formatter to run 'imports-formatter .' in project root, to format your import blocks, mentioned in CONTRIBUTING.md above)
[ ] Your new-created file needs to have apache license at the top, like other existed file does.
[ ] All integration test passed. You can run integration test locally (with docker env). Clone our dubbo-go-samples project and replace the go.mod to your dubbo-go, and run 'sudo sh start_integration_test.sh' at root of samples project root. (M1 Slice is not Support)

Codecov Report

Merging #30 (5ecfd19) into master (7bc593b) will increase coverage by 0.12%.
The diff coverage is 11.62%.

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
+ Coverage   46.78%   46.90%   +0.12%
==========================================
  Files         298      298
  Lines       17163    17178      +15
==========================================
+ Hits         8029     8058      +29
+ Misses       8285     8265      -20
- Partials      849      855       +6

Impacted Files
Coverage Δ

protocol/dubbo/dubbo_codec.go
1.75% <0.00%> (-0.13%)
:arrow_down:

protocol/dubbo/impl/codec.go
41.66% <0.00%> (-0.55%)
:arrow_down:

remoting/getty/readwriter.go
39.28% <27.27%> (-3.31%)
:arrow_down:

remoting/getty/listener.go
26.92% <50.00%> (ø)

metrics/prometheus/reporter.go
28.57% <0.00%> (+1.71%)
:arrow_up:

filter/metrics/filter.go
100.00% <0.00%> (+15.00%)
:arrow_up:

metadata/report/delegate/delegate_report.go
42.38% <0.00%> (+15.89%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7bc593b...5ecfd19. Read the comment docs.


Migrate from MUI to Yorkie-UI for Yorkie project

What would you like to be added:
Currently, the project is utilizing MUI for the UI components. It is necessary to migrate to Yorkie-UI in order to better align with the overall project architecture and styling requirements.
Why is this needed:
The migration to Yorkie-UI will ensure consistency in design patterns and UI components across the project, providing a more streamlined development experience and maintaining a cohesive visual identity.
Additional Information:

Yorkie-UI npm package: Yorkie-UI npm

We decided to maintain MUI.


Normalize let/const exports in vscode.d.ts

Some properties in extension API like https://github.com/microsoft/vscode/blob/0dfcf2c82934ef40ed584d3d6a1764f0c0d78069/src/vscode-dts/vscode.d.ts#L10349 are exported with let because they can change, but this also implies that extensions can set them. That isn't allowed, only vscode can change them. Some similar properties already use const. We should normalize these to const everywhere to avoid confusion.
readonly communicates the intent a little better, but this is a namespace, not properties on an interface. But I think it's clear from the name and behavior that these properties will change.

We'd talked about making the namespaces an interface or a type. type doesn't support merging, like from proposals. So we can have something like this:
export interface Window {

	/**
	 * Represents the grid widget within the main editor area
	 */
	readonly tabGroups: TabGroups;

	/**
	 * The currently active editor or `undefined`. The active editor is the one
	 * that currently has focus or, when none has focus, the one that has changed
	 * input most recently.
	 */
	readonly activeTextEditor: TextEditor | undefined;

	/**
	 * The currently visible editors or an empty array.
	 */
	readonly visibleTextEditors: readonly TextEditor[];

	/**
	 * An {@link Event} which fires when the {@link window.activeTextEditor active editor}
	 * has changed. *Note* that the event also fires when the active editor changes
	 * to `undefined`.
	 */
	readonly onDidChangeActiveTextEditor: Event<TextEditor | undefined>;
...
}

export const window: Window;

It's not a problem for the type to have the same name as the interface, so it could also be export const window: window; but that looks weird, and we capitalize other interfaces.


Fix typo and style issues in lmp-customization

Addressed issues to more closely adhere to the style-guide.
Checked rendered output, and ran linter. No issues to report.
No associated ticket, minor fix.
PR Template and Checklist
Please complete as much as possible to speed up the reviewing process.
Readiness and adding reviewers as appropriate is required.
All PRs should be reviewed by a technical writer/documentation team and a peer.
If effecting customers—which is a majority of content changes—a member of Customer Success must also review.
Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist

[x] Run spelling and grammar check, preferably with linter.
[x] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[x] Review for wordiness
[x] Match tone and style of page/section.
[ ] Run make linkcheck.
[x] View HTML in a browser to check rendering.
[x] Use semantic newlines.
[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

[x] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for cd6f7ddfafcd20f1eed696543fcf23742f9ac876 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2667/docs/artifacts/html/index.html

Docs for b9c09a94d156059e3ec6f5cbfa965b5c10de0b69 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2669/docs/artifacts/html/index.html


compactor/sidecar: HostUnusualDiskWriteLatency

Hello
When compactor have works to do, or Thanos sidecar is uploading the data to MinIO, there's high volume of disk write operations in a small window of time
When it occurs, this alerts is fired:
          - alert: HostUnusualDiskWriteLatency
            expr: rate(node_disk_write_time_seconds_total[1m]) / rate(node_disk_writes_completed_total[1m]) > 0.1 and rate(node_disk_writes_completed_total[1m]) > 0
            for: 2m
            labels:
              severity: warning
            annotations:
              summary: Host unusual disk write latency (instance {{ $labels.instance }})
              description: "Disk latency is growing (write operations > 100ms)\n  VALUE = {{ $value }}\n  LABELS: {{ $labels }}"
          - alert: KubernetesVolumeFullInFourDays
            expr: predict_linear(kubelet_volume_stats_available_bytes{persistentvolumeclaim!~".*prometheus.*|.*elasticsearch.*"}[6h], 4 * 24 * 3600) < 0
            for: 0m
            labels:
              severity: critical
            annotations:
              summary: Kubernetes Volume full in four days (instance {{ $labels.instance }})
              description: "{{ $labels.namespace }}/{{ $labels.persistentvolumeclaim }} is expected to fill up within four days. Currently {{ $value | humanize }}% is available.\n  VALUE = {{ $value }}\n  LABELS: {{ $labels }}"

Would be nice to see some throttling mechanism to mitigate intensive operations (chunks with buffer/+sleep between chunks or some kind..)

I wonder if there are appropriate host/node level mechanisms to apply instead of something on application level? (:


Commands don't work in the content of the page option

Describe the bug
Commands don't work in the content of the page option.
Additional information

TeX distribution: MacTeX 2024
Package citation-style-language version: latest release
LaTeX engine: lualatex

To Reproduce
\documentclass{article}

\begin{filecontents}[overwrite, noheader]{\jobname.json}
[
    {
        "id": "King2008-Gramma",
        "type": "thesis",
        "author": [
            {
                "family": "King",
                "given": "John T."
            }
        ],
        "title": "A Grammar of Dhimal",
        "publisher-place": "Netherlands",
        "publisher": "Universiteit Leiden",
        "URL": "https://scholarlypublications.universiteitleiden.nl/handle/1887/13072",
        "genre": "doctoral dissertation",
        "issued": {
            "date-parts": [
                [
                    "2008",
                    "9",
                    "3"
                ]
            ]
        },
        "accessed": {
            "date-parts": [
                [
                    "2024",
                    "08",
                    "13"
                ]
            ]
        }
    }
]
\end{filecontents}

\usepackage{citation-style-language}
\cslsetup{style = turabian-fullnote-bibliography}
\addbibresource{\jobname.json}

\newcommand{\hlm}{hlm.}

\begin{document}

Commands don't work in the content of the page option.\cite[page = {PDF \hlm\ 20}]{King2008-Gramma}

\end{document}

Screenshots

Turabian CSL

This bug is fixed in 381f661.
BTW what does “hlm.” mean?

Thank you. The abbreviation "hlm." means "halaman" ('page' in Indonesian).


Response map crash

version 15.0.0
This crash problem has troubled me for a long time. First of all, this problem is not necessary, but his voice and shadow can always be found in the crash log of each version, which makes me very headache. Can anyone give me some help!
0 libobjc.A.dylib objc_msgSend + 8
1 CoreFoundation NSDictionaryI_new_block_invoke + 96
2 CoreFoundation __NSDictionaryI_new + 480
3 Foundation newJSONObject + 1400
4 Foundation newJSONValue + 496
5 Foundation newJSONValue + 1044
6 Foundation newJSONObject + 568
7 Foundation newJSONValue + 496
8 Foundation newJSONObject + 568
9 Foundation newJSONValue + 496
10 Foundation -[NSJSONReader parseData:options:error:] + 1096
11 Foundation closure #1 in JSONDecoder.decode(:from:) + 128
12 Foundation JSONDecoder.decode(:from:) + 464
13 Foundation dispatch thunk of JSONDecoder.decode(:from:) + 56
14 Moya Response.swift - 第 181 行 Response.map(:atKeyPath:using:failsOnEmptyData:) + 181
15 XXXX  - 第<PHONE_NUMBER> 行 specialized XXXPlugin.process(:target:) +<PHONE_NUMBER>

Hello, I got the same problem, crash at Moya > Response.swift line 181. do you have any suggestions? Thank You


StackShare account for ChicagoBoss

Any plans to create an account for ChicagoBoss on StackShare, similar to the one of the Phoenix Framework?

not a first priority. but will do it

made a try, no luck, closing


Fixed animation playReverse with yoyo flag

Fixed a bug found by @Ben-Millions (#3837) that after "playReverse" implementation, the yoyo flag run the animation infinitely.
Fixed playReverse repeat, now playReverse respect the repeat number.
I tested it with @Ben-Millions sample  and my sample in phaser3 examples.

Thanks for the quick turn-around on this.


Drop-down link to v1.5 (latest) from v1.4 docs does not work.

Describe the bug
Drop down link to v1.5 (latest) from v1.4 docs does not work.
Steps to reproduce

Navigate to https://v1-4.docs.dapr.io/ or its subpages.
In the navigation bar, click the version drop-down and select v1.5 (latest)
Link leads to https://v1-4.docs.dapr.io/# instead of v1.5 docs

Expected behavior
Link should point to latest docs (https://docs.dapr.io)
Screenshots

Desktop (please complete the following information):

OS: Win10
Browser Edge Dev
Version 97.0.1072.8

Additional Context
Issue is specific to v1.4 version of the docs, earlier versions will point to latest https://docs.dapr.io (even if the expected number version is outdated in the drop-down for which is latest)

fixed


TII-201 fix isUrlAccessed field updating, modify logic to allow or deny serving the file

https://jira.sakaiproject.org/browse/TII-201
There is a new contentreviewitem property called isUrlAccessed which controls whether or not a submission in the queue (file) can be retrieved by Turnitin. Before granting access to the file resource (in BaseAssignmentService.getHttpAccess()), it checks this property and returns early if it is true.
After many, many iterations of testing and debugging, it seems that there's a race condition happening here.
The TII callback to retrieve the file happens very quickly, and sets the isUrlAccessed to true. Milliseconds later, the queue job continues on, and does some unrelated updates to the same contentReviewItem. Now, because the queue job is using the DAO to update the object, regardless of if it's explicitly setting the isUrlAccessed attribute or not, Hibernate will update this field to the default value.
So, TII gets the file, sets the isURLAccessed to true, then the queue job continues and sets it almost immediately back to false. In effect, this leaves the URLs to access these content review files open for anyone to access once more, as there is no authentication performed on these requests.
The solution to this, is marking the urlAccessed column as insert=true and update=false in the Hibernate mapping XML file. In this way, when creating a new contentReviewItem, the urlAccessed parameter will default to false, and any code updating this object via the DAO will reject any changes to this column.
Then, in the one and only place we want to set the urlAccessed value to true (after TII has called us back and retrieved the file), we call a method that runs some custom HQL to manually update the value to true for the given row.
We've also added some extra obfuscation to these URLs because they are not authenticated. Currently, the URL could technically be determined by an attacker (with a combination of sleuthing/inspecting elements and brute force), as it contains the site ID, the submission ID, and the (positive, sequential integer) ID of the content review item.
The extra protection we've implemented, is by appending the hash value of the contentId to the access URL. Due to the fact that the content ID is a combination of two additional, non-user facing UUIDs and the actual file name and extension of the file in question, it is near impossible for an attacker to determine this string and it's subsequent hash code. When the URL is accessed, it will first check to see that the hash codes match, and short circuit if not.
Lastly, we've modified the conditional logic around actually serving up the file. Previously, the file was only served if and only if the isUrlAccessed property was false. However, due to the calls to and from TII being asynchronous, we cannot safely assume that the status of the item will be either 1 or 2 by the time TII attempts to retrieve the file. Depending on timing, it could very well be one or the other (coupled with the fact that we've seen TII make multiple requests to get the same file), and we need to allow TII to get the file in both of these circumstances.
Therefore, we've determined that the file should be served in only the following circumstances:

if the URL has not yet been accessed
if the URL has been accessed, but the status is one of the following: 1 (not yet submitted), 2 (submitted, waiting for report), and 4 (submission error, will retry)

Of course, the hashing protection and validation happens prior to checking these conditions, serving as both another layer of obfuscation as well as a short circuit for the process.

This is directly tied to https://github.com/sakaicontrib/sakai-turnitin/pull/6

This doesn't work for us as we have a case sensitive MySQL and the content review tables are all in upper case for us.

Interesting, I tested on both MySQL and Oracle with success. I suppose my MySQL instances are all lower case then, however in Oracle everything is upper case and still works.
Would changing the table and column names to upper case fix the issue for you? Would it then still work on non-case sensitive MySQL instances?

Yep, having it all it upper case should work.

Okay, thanks for the feedback @buckett, I'll get to work on an amendment PR.

I think if you use HQL then hibernate worries about this, but yep, upper case should also work.

Created #36 to address this


Rc1 fixes p1

#264: just logging error when event level is known, not throwing
#263: changing the userInfo response type to string on OpenIdConnectProtocolValidationContext
#251: crypto extensibility for hash validation
#265: fixing log message code
#240: split logmessages.cs file per project

[ ]   File: src/Microsoft.IdentityModel.Protocols.OpenIdConnect/OpenIdConnectProtocolValidator.cs:L444-473
 Let's remove support for SHA1, it is not a valid algorithm for JWS.
https://tools.ietf.org/html/rfc7518#section-3
[ ]   File: src/Microsoft.IdentityModel.Protocols.OpenIdConnect/OpenIdConnectProtocolValidator.cs:L444-473
 We need to support alg values, PS256, PS384, PS512.  We should augment SecurityAlgorithms with three new values and support them here. Since this only checks the 'hash', even though we may not have support for PS256 (right now) if users have extended crypto, then this would fail and they would have to extend here also to make that scenario supported.
[ ]   File: src/System.IdentityModel.Tokens.Jwt/LogMessages.cs:L1-91
 add newline
[ ]   File: src/System.IdentityModel.Tokens/SecurityAlgorithms.cs:L59-67
 add PS256, PS384, PS512
[ ]   File: src/Microsoft.IdentityModel.Protocols.OpenIdConnect/LogMessages.cs:L1-104
 newline
[x]   General Comment
 @brentschmaltz
 @brentschmaltz updated
[x]   File: src/Microsoft.IdentityModel.Protocols.WsFederation/LogMessages.cs:L1-41
 nice in splitting these up :+1:
new line

This pull request is now managed with CodeReviewHub. Refresh.

@brentschmaltz

@brentschmaltz updated

:shipit:


0.7.0 update

0.7.0 came out since you started the original recipe.
The wheel files now include COPYING, but not the source tarballs yet: https://github.com/althonos/fs.sshfs/issues/5.

I just released v0.7.1 including your MANIFEST as well.

@dougalsutherland while 0.7.1. is now merged I recognized there is no  appveyor build done.
I had removed the selector beforehand. Why is the windows build missing?


Chrome extension: Cannot save bookmarks with empty title or description

Environment:
Linkding version: 1.36.0 (latest)
Chrome version: 129.0.6668.90
Extension version: 1.12.0
Issue Description:
When using the linkding Chrome extension, there's a bug with saving bookmarks. If the initial title or description fetched from the webpage is empty, attempting to save results in an error stating that the title or description cannot be null. However, if I input random content and then delete it, I am able to save the bookmark successfully.
Expected Behavior:
It should be possible to save bookmarks even when the title or description fields are empty.
Additional Information:
This issue seems to have appeared within the last month; it was functioning correctly before.

That was fixed just yesterday: https://github.com/sissbruecker/linkding-extension/pull/86
Still needs to be released.


Update nfp library and release new version

Hi,
Just opening an issue to track adding a version of the nfp library with this fix into excelize and release a new version we can use in our internal services.
EDIT: I see that the library was already updated, so I'll use the latest commit for now. Still, a new release tag would be nice.

Thanks for your contribution. This library usually releases a stable version every 3 or 4 months. These changes will be released in the next version. Please use the Go module to lock the version before the new version is released.


Duplicated templates

I can't repro:

Microsoft Visual Studio Enterprise 2017  (15.4)
Version 15.5.0
VisualStudio.15.Release/15.5.0+27130.0
Microsoft .NET Framework
Version 4.7.02558

Installed Version: Enterprise

Visual Basic 2017   00369-60000-00001-AA818
Microsoft Visual Basic 2017

Visual C# 2017   00369-60000-00001-AA818
Microsoft Visual C# 2017

Visual F# 4.1   00369-60000-00001-AA818
Microsoft Visual F# 4.1

Application Insights Tools for Visual Studio Package   8.10.01106.1
Application Insights Tools for Visual Studio

ASP.NET and Web Tools 2017   15.0.31120.0
ASP.NET and Web Tools 2017

ASP.NET Core Razor Language Services   1.0
Provides languages services for ASP.NET Core Razor.

ASP.NET Web Frameworks and Tools 2017   5.2.51007.0
For additional information, visit https://www.asp.net/

Azure App Service Tools v3.0.0   15.0.31106.0
Azure App Service Tools v3.0.0

Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.

JavaScript Language Service   2.0
JavaScript Language Service

Microsoft Azure Tools   2.9
Microsoft Azure Tools for Microsoft Visual Studio 2017 - v2.9.51120.3

Microsoft Continuous Delivery Tools for Visual Studio   0.3
Simplifying the configuration of continuous build integration and continuous build delivery from within the Visual Studio IDE.

Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines

Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers

Microsoft Visual Studio Tools for Containers   1.1
Develop, run, validate your ASP.NET Core applications in the target environment. F5 your application directly into a container with debugging, or CTRL + F5 to edit & refresh your app without having to rebuild the container.

NuGet Package Manager   4.5.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit http://docs.nuget.org/.

Azure App Service Tools v3.0.0   15.0.31121.0
Azure App Service Tools v3.0.0

C# Tools   2.6.0-beta3-62405-07. Commit Hash: 4939752b756ad32cb2ce1bfeab1d4f1637966e08
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.

JavaScript Language Service   2.0
JavaScript Language Service

Microsoft Continuous Delivery Tools for Visual Studio   0.3
Simplifying the configuration of continuous build integration and continuous build delivery from within the Visual Studio IDE.

Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines

Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers

NuGet Package Manager   4.5.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit http://docs.nuget.org/.

SQL Server Data Tools   15.1.61710.120
Microsoft SQL Server Data Tools

TypeScript Tools   15.5.11025.1
TypeScript Tools for Microsoft Visual Studio

Visual Basic Tools   2.6.0-beta3-62405-07. Commit Hash: 4939752b756ad32cb2ce1bfeab1d4f1637966e08
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio

Did you install nightlies or something on top of this?

Ah, it's on an exp instance. No idea if this is ok. No duplicates on vanilla 15.6

😕 Bizarre. We'll have to make sure this doesn't slip through somehow.