Some observations / possible optimisation

Thank you for this nice implementation of SSS!
I'd like to bring a few issues to your attention:

Your choice of prime excludes 189 secret values in the range (0..2^256-1). Now, chances are tiny that any slice of s is >= your prime, but how about checking, or slicing by p, instead of 2^256, just in case?

What if my random secret just happens to have trailing zeros?  Would those not be lost in mergeIntToByte()? I guess that warrants at least a documentation, or maybe encoding the length of the secret in some form.

Thank you for not reusing the polynomial coefficients a_1 .. a_{k-1} across the different pieces of large secrets. That would have ruined the day :-)

Suggestion: In the literature, people use x=1..n without adverse effects. This would almost halve the size of your output.

Putting these points up for discussion here. Do you think they are valid, and might warrant a code change?
Germano

Yeah, we're not actively working on this project any more. There's several other projects in this space like hashicorp vault that we'd probably use instead.
If you wanted to improve the libraries, feel free. I don't know if we should update all libraries to match (@Mab879 might have some opinions here), or if we're content with letting them be incompatible.

(iirc) We chose the first prime smaller than 2^256 so it would fit in a fixed 32 bytes each. You could slice by p, but that'd require a bit of work to rebuild the secret afterwards; IIRC this scheme lets you reconstruct bits of the secret if you wanted to, without rebuilding the entire secret. You could slice by 2^248 I suppose... We didn't really have a great solution to this and mostly chose to ignore this problem :)
Yes, we should really use a real padding scheme. You could definitely encode 4-bytes of length if you wanted to.
Mhmm, we at least tried to do that correctly. :)
(I haven't looked at the code in a long while); do you have a reference for this? I'm not immediately sure what you're talking about here.


Brunch cannot include modules using symbolic link

Hi,
I think this is the case the Brunch should support:
cd node_modules
ln -s ../web/static/js app

In a module:
require('app/lib/myModule');

And it failed to load myModule:
error: Processing of node_modules/app/lib/myModule.js failed. Cannot find module '[...]/package.json'

When using node REPL, it's fine to load app/lib/myModule
Environment

Brunch: 2.9.1
Node: 6
NPM: v3
Operating system: OSX 10.11

Thanks for reaching out!
Seems like this issue might be related to #1593.

Closed in favor of #1593.


tapToClose prop can glitch animation if you drag a really short distance

Here is a preview of what I'm talking about:

It's dropped some frames (it glitches back and forth more times than once) but the gif still shows it.

Yes, I put in a little naive time delta check to try and avoid this, but clearly that is not enough. I think a better approach would be to check the distance travelled rather than the tap duration.
Question: intuitively do you expect the drawer to open or close in this situation where tapToClose=true but a short drag was detected?

Intuitively I feel open. But that depends on the distance travelled. It's a tough one. If it's really really short then it shouldn't open, but it also shouldn't count as a tap because distance was travelled. I think taps don't travel any distance.

hey dubert, i am facing another first time tap stutter and drawer doesn't close. i created an issue https://github.com/rt2zz/react-native-drawer/issues/51
can you check in your code such behaviour? Also can you pl review my code pasted in the issue on how i am navigating to different views on menu taps.
thanks-in-advance.

fixed. a tap is now considered when the distance travelled is < 50 and the time travelled is < 350.
Additionally when a "tap" is detected no pan events are triggered (thus avoiding the conflicting actions originally observed)


ascii-art problem

Hi, i have problem, when install dependencies. I can't install ascii-art module, becouse it require MS BUILD tools. Please remove this dependency.

I want to help develop this framework.

Hi.

becouse it require MS BUILD tools

why? what is MS BUILD tools?

I want to help develop this framework.

thats great, start with ideas what you want to implement in this framework

feel free to reopen it if you are still interested in this issue.


Investigate how to document keyboard and mouse gestures on macOS

Check how Control/Command keyboard shortcuts are documented in other cross-platform software (Adobe, Microsoft, etc.) so that we document these differences in shortcuts similarly in Slicer.
Differences include:

Control/Command key modifier is swapped on macOS compared to Windows and Linux
Right-click on macOS is often available as Ctrl-Left-click
There is no Alt key on macOS, it is often mapped to Option

In many cross-platform software programs, notably those from Adobe and Microsoft, keyboard shortcuts are generally documented to reflect the conventions of the respective operating systems, while maintaining consistency wherever possible.

[ ] General information about keyboard shortcut documentation in Adobe products like Photoshop, Illustrator, InDesign:

1 Key Modifiers:

Windows: Ctrl (Control) is commonly used as the primary modifier. For example, Ctrl+C for copy.
macOS: Command (⌘) is typically the main modifier. For instance, Command+C for copy.

2 Right-Click:

Windows: Right-clicking is standard and does not require an additional modifier.
macOS: Right-clicking can be simulated by clicking with the Ctrl key held down or using two fingers to click on a trackpad.

3 Alt/Opt Key:

Windows: The Alt key is often used as a modifier in keyboard shortcuts.
macOS: The Alt key is usually mapped to the Option key. Alt+C on Windows might be equivalent to Option+C on macOS.

4 More about the default keyboard shortcuts: link here

[ ] General information about keyboard shortcut documentation in Microsoft products such as Outlook, Teams, Excell:
1 Key Modifiers:

Windows: Ctrl (Control) is commonly used as the primary modifier. For example, Ctrl+C for copy.

macOS: On Microsoft products running on macOS, the Command key (⌘) is typically used as the main modifier.

2 Right-Click:

Right-clicking is a standard action on both Windows and macOS and is often used to access context menus.

3 Alt/Opt Key:

The Alt key is often used as a modifier in keyboard shortcuts on Windows.
On macOS, the Alt key is equivalent to the Option key.

4 More about the default keyboard shortcuts:
General link here
Outlook link here
Note that all Microsoft keyboards are equipped to perform shortcuts in Windows and other Microsoft applications.

Describe both Windows and Mac everywhere:

Adobe products: https://helpx.adobe.com/photoshop/using/create-documents.html
OnShape: https://cad.onshape.com/help/Content/shortcut_keys.htm
Chrome: separate table for Windows&Linux, Mac - https://support.google.com/chrome/answer/157179?hl=en&co=GENIE.Platform%3DDesktop#zippy=

Only describe Control:

Microsoft products: https://support.microsoft.com/en-us/topic/keyboard-shortcuts-f2302f54-02a3-a62b-36ef-74ee5a83cb87
GIMP: https://docs.gimp.org/quickreference/gimp-keys-en.pdf
Blender: https://docs.blender.org/manual/en/latest/interface/keymap/introduction.html

Visual studio code provides different cheatsheets for each OS:
https://code.visualstudio.com/docs/getstarted/tips-and-tricks#_keyboard-reference-sheets
For their online documentation it seems they embed a code snippet that dynamically changes the document based on what OS your browser is coming from:
https://github.com/microsoft/vscode-docs/blob/main/docs/getstarted/tips-and-tricks.md?plain=1#L80
If we think it's worth it we could do something similar with readthedocs.

re: vscode documentation

They are indeed processing the kb(shortcut-identifier) to generate some javascript dynamically updating the page based on the current user agent and the bindings organized in json files^1.
That said, after inspecting the microsoft/vscode-docs repository, it looks like the corresponding infrastructure is organized in a private project called microsoft/vscode-website^2

getstarted/keybindings#_basic-editing

https://code.visualstudio.com/dist/index.js

According to the official documentation, here is the use of the Control key in various software on Windows and Mac:
Adobe Products:

Windows:

In Adobe products on Windows, the "Control" key is often used in combination with other keys for various shortcuts. For example, "Ctrl+C" is used for copy, "Ctrl+V" for paste, and so on.

Mac:

On Mac, the equivalent modifier key is "Command" (⌘). So, commands like "Command+C" and "Command+V" are used for copy and paste.

OnShape:

Windows:

In OnShape on Windows, the "Control" key is used in combination with other keys for keyboard shortcuts. Check the provided link for a detailed list.

Mac:

On Mac, similar functions might use the "Command" key instead.

Chrome:

Windows/Linux:

In Chrome on Windows and Linux, the "Control" key is commonly used for various shortcuts. For instance, "Ctrl+N" for a new window, "Ctrl+T" for a new tab, etc.

Mac:

On Mac, the equivalent would be "Command" key. So, "Command+N" for a new window, "Command+T" for a new tab.

Microsoft Products:

Windows:

In Microsoft products on Windows, the "Control" key is widely used for shortcuts. For example, "Ctrl+S" for save, "Ctrl+Z" for undo, etc.

Mac:

On Mac, these functions typically use the "Command" key.

See comments above on documentation provided by VS Code!!

GIMP:

Windows:

In GIMP on Windows, the "Control" key is used in combination with other keys for various shortcuts.

Mac:

On Mac, the "Command" key is used for similar functions.

Blender:

Windows:

In Blender on Windows, the "Control" key is often part of keyboard shortcuts.

Mac:

On Mac, the "Command" key would be used for equivalent functions.


Add 1.20 for Prowjob Liner

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/retest

/retest

/lgtm


MINIFICPP-1686 - Processor destructors are not called

Not just processors were affected, also connections and process groups.
Using unique_ptrs instead of shared ones in many places, preventing circular dependencies.
Also there is an integration test to ensure destructors will be called from now on.
Some header rationalization also took place in the Processor base class, affecting subclasses.
Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

You may want to mark MINIFICPP-839 (or your newer issue) as duplicate. There were similar efforts in #563, but it was eventually abandoned for some reason.

You may want to mark MINIFICPP-839 (or your newer issue) as duplicate. There were similar efforts in #563, but it was eventually abandoned for some reason.

Done, also marked it as resolved, to be continued here.


problem with evaluateSegmentationClassificationDir(dirName, modelName, methodName)  from audioSegmentation.py

Error in computePreRec! Confusion matrix and classNames list must be of the same size!
Traceback (most recent call last):
File "C:\Program Files (x86)\JetBrains\PyCharm Community Edition 2016.2.3\helpers\pydev\pydevd.py", line 1580, in
globals = debugger.run(setup['file'], None, None, is_module)
File "C:\Program Files (x86)\JetBrains\PyCharm Community Edition 2016.2.3\helpers\pydev\pydevd.py", line 964, in run
pydev_imports.execfile(file, globals, locals)  # execute the script
File "E:/int/pyAudioAnalysis-master/recognizerEmotionInVoice.py", line 76, in
testSegmentationEvaluation(select_folderEmotion,selected_types[numModel],selected_models[numModel])
File "E:/int/pyAudioAnalysis-master/recognizerEmotionInVoice.py", line 69, in testSegmentationEvaluation
aA.segmentationEvaluation(dirName, modelName, methodName)
File "E:/int/pyAudioAnalysis-master\audioAnalysis.py", line 212, in segmentationEvaluation
aS.evaluateSegmentationClassificationDir(dirName, modelName, methodName)
File "E:/int/pyAudioAnalysis-master\audioSegmentation.py", line 580, in evaluateSegmentationClassificationDir
[Rec, Pre, F1] = computePreRec(CM, classNames)
TypeError: 'NoneType' object is not iterable

Hi Dagny - Im hitting the same error. What was the resolution to this?


Change routing build to t-route master branch

Switch to building/installing t-route master instead of t-route ngen branch.
Additions

Several small build system improvements, including using ${BUILD_PARALLEL_JOBS} instead of $(nproc), and a way to optionally pass a pre-downloaded boost tarball to the image creation process.

Removals

Does not work out of the box to use legacy t-route (e.g. ngen branch), which might limit capability with some older hydrofabric datasets.

Changes

Now compiles and builds master t-route by default (instead of ngen branch)
Several Python elements are now installed in the global (root) site. This enables use-cases where a user other than mpi(1000) is used, e.g. using docker run --user=$(uid -u), so now the routing modules are available to that user (tested!).

Testing

Built and ran the gauge_01073000 dataset, including using --user to change the uid

Screenshots
Notes

Todos

Checklist

[X] PR has an informative and human-readable title
[X] Changes are limited to a single goal (no scope creep)
[X] Code can be automatically merged (no conflicts)
[X] Code follows project standards (link if applicable)
[X] Passes all existing automated tests
[X] Any change in functionality is tested
[X] New functions are documented (with a description, list of inputs, and expected output)
[X] Placeholder code is flagged / future todos are captured in comments
[X] Visually tested in supported browsers and devices (see checklist below :point_down:)
[X] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[X] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Target Environment support

[X] Linux

@mattw-nws, pinging you again on this one, just to keep it on your radar.  In addition to the stuff I commented on, it looks like the branch will need rebasing, probably due to #353.

This is now superseded by #429.


:tada: use @deriveTraverse annotation in our ADTs

Waiting for droste to release https://github.com/andyscott/droste/pull/90
This means that we don't need to derive the functor ourselves.
Also, since we'll have Traverse instance instead of only Functor, we'll be able to use monadic recursion schemes that place a Traverse restriction on the pattern functor.

Geez, could you put that first headline in bigger/bolder text? 😄


page information is incorrect based on actual usage.

This page shows that the layout should include the vendor ID in the first column.  However, in use, the first column is the distribution.  You can't paste without entering the vendor ID first.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4f28e3eb-a698-1d38-6f40-a70ca60358ae
Version Independent ID: fa84b2cc-b50d-a527-bd5d-e3d37ba12451
Content: Copy/Paste Purchasing transactions from Excel - Dynamics GP
Content Source: dynamics-gp/whats-new/copypaste-purchasing-transactions-from-excel.md
Product: dynamics-gp
GitHub Login: @theley502
Microsoft Alias: theley

Thanks for the feedback! This has been addressed in the help.


General: Fix SAML2 Login

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context
Fixes #5710
Description
This basically reverts #5604.
Interestingly, the Bamboo WAR artefact was broken with the mentioned PR. However, local builds in the regular source directory, local builds in clean environments, and WARs/Docker images produced by Github Actions continued to work at least until release 5.10.2 but were broken in the next release 5.11.0.
An additional investigation why the change continued to work in some environments is necessary so that the deprecated security configuration can be replaced. The newly introduced integration test that checks that the redirect works as intended also passes for the actually broken configuration.
Therefore, this PR is to be seen as an intermediary hotfix to allow us to continue upgrading our systems until the underlying cause is found.
I checked that the produced CI artefacts are now working again:

Bamboo WAR with local Docker build
Docker image as built in Github Actions

Additionally, the base image for Docker was updated, as the openjdk image is deprecated. The last update for the previous image has been released six months ago and therefore not received any security updates since.
The new eclipse-temurin:17-jre image is also smaller by avoiding to include the full JDK when not needed for running the application (531MB vs 720MB before).
Steps for Testing

Prerequisites:

System with SAML2 set up

Login via Shibboleth/SAML2.

Review Progress
Code Review

[x] Review 1
[ ] Review 2

Manual Tests

[x] Test 1
[ ] Test 2

Test Coverage
unchanged

Okay, I’ve now got some weird behaviour:
If I build the WAR and Docker image from this branch locally and deploy it, it works. When I use the Docker image built in GitHub Actions here from the same commit, it does not.

Current develop branch when built locally works, too.

Have you tested what happens if you use the War from the release and build the docker image locally ?

Good idea. Now I did with the WAR from https://github.com/ls1intum/Artemis/releases/tag/5.11.2: It’s broken, too.

Ok then I guess, we can assume that the broken jar (actions build) is the problem.

E.g., we could try to test whether the build (actions) works with temurin instead of Zulu

I already tried that: https://github.com/ls1intum/Artemis/pull/5721/commits/63b77fcf22e81384c58d1776178e7b4644b3dd0a

We could also try to build Artemis within the docker image in a build stage. Then we would not directly rely on actions.

I now made sure the CI run uses Temurin and the most recent Node/npm versions. The only difference between the WAR files is now some metadata like build time and the name of JS files.
Otherwise all class files are identical (verified by md5hash):
1605c1603
< fbbf87b93a8d3c2ae505d70b878c6d0b  WEB-INF/classes/git.properties
---
> 06ced52695f471d0652d050269cd60c2  WEB-INF/classes/git.properties
1610c1608
< a8406d996477822cd1d3e1297553bd45  WEB-INF/classes/META-INF/build-info.properties
---
> be739e9f46fffcbc915d27799bdf6b59  WEB-INF/classes/META-INF/build-info.properties
1726,1727c1724,1725
< f6162e709ee57faab74475e7a080ca03  WEB-INF/classes/static/1326.0cf362dff96b19f7.js
< e5bdf728d15ef86d00ff3cf6f2079010  WEB-INF/classes/static/1326.0cf362dff96b19f7.js.map
---
> c2de2b7cf08fc5237e1a633a4b83b50f  WEB-INF/classes/static/1326.4661e29918019082.js
> e1a9904e0883151ffe78c04864bc6d5e  WEB-INF/classes/static/1326.4661e29918019082.js.map
1779,1780c1777,1778
< dc5da9819ee415e2b974292047682743  WEB-INF/classes/static/4502.c917e89a7331bc14.js
< 5e6f66f7f1df8ea757846d1f8a3e9f18  WEB-INF/classes/static/4502.c917e89a7331bc14.js.map
---
> ba466ceb9874f7fcd1227fe7317695c4  WEB-INF/classes/static/4502.edeb8f96f49dabe7.js
> d5e74b1c2b588efc8f19153bbe801a9a  WEB-INF/classes/static/4502.edeb8f96f49dabe7.js.map
1918c1916
< bf955ba4968dc591ffc7b77370ec0b53  WEB-INF/classes/static/index.html
---
> 209aa3e23465e1386904337715dcc8bd  WEB-INF/classes/static/index.html
1989,1990c1987,1988
< f028c2c2f4605560ecd8f056060cb283  WEB-INF/classes/static/main.711ac3295180a8b5.js
< ac35e9632486e2213747222e32acb87d  WEB-INF/classes/static/main.711ac3295180a8b5.js.map
---
> 676695d54d79dcd1c283e07c25d81da4  WEB-INF/classes/static/main.a817fdb0b82d256f.js
> 14045c842217204964abd629decb6cd6  WEB-INF/classes/static/main.a817fdb0b82d256f.js.map
1992c1990
< 2f0e030e720cee3f61478e6e2bd8c6df  WEB-INF/classes/static/ngsw.json
---
> 2ad53ed249a670f379e4639a2bdb3868  WEB-INF/classes/static/ngsw.json
1997,1998c1995,1996
< 8e7e95860e7373aa198e29b30973ed01  WEB-INF/classes/static/runtime.cc3d9795627939f3.js
< 3aa8f9b385113e8003409c8c0675f642  WEB-INF/classes/static/runtime.cc3d9795627939f3.js.map
---
> 004d7c74a3bec34d24e2428b50194486  WEB-INF/classes/static/runtime.5bb1b4b09556582e.js
> 584d4324ca680c4f310c0383caf8c6b1  WEB-INF/classes/static/runtime.5bb1b4b09556582e.js.map
2006c2004
< c9f878275475ce8aaa067f59210755b6  WEB-INF/classes/stats.html
---
> a9d8c8b36a9f9e8224c1420674c00c43  WEB-INF/classes/stats.html
2385c2383
< 2cd3533ceb1a4676f11891cce2501a48  WEB-INF/classpath.idx
---
> c738f870515887ee27e036190c42f6ca  WEB-INF/classpath.idx

The differences in WEB-INF/classes/static/ngsw.json are only the different names for the *.js files. In classpath.idx the order is a bit different. In the js files I’ve looked into the only differences seem to be identifier names.

I'll check whether it works now asap :)

Oh, I forgot that part in the comment above: It still doesn’t work for me.

@b-fein I've tested the following:

Build via Docker eclipse-temurin:17 (linux amd64) -> Does not work
Build on Windows 11 eclipse-temurin 17 -> works

Did you created your working WAR on Windows as well?

No, I’m using a Linux machine.

Ok, then I do not get what's different .. Have you also tried building your branch directly in docker (that's what I did and what didn't worked)
On windows I used "Temurin-17+35"
The docker contains "Temurin-<IP_ADDRESS>+1"

maybe we have a cached dependency ?

I have now tried

building the WAR locally inside a eclipse-temurin:17 image with a completely fresh repo clone and no mounted directories. ⇒  works
updating the base image from the deprecated openjdk (last update five months ago for the used version) to the same temurin image as above. ⇒  works locally, broken when built in GitHub Actins
clearing the GitHub Actions cache and rebuilding ⇒  still broken

Interesting .. I've also build the branch with eclipse-temurin:17 in docker .. that didn't work for me (as mentioned before) ->I'll test that again. (Did you build it as root or as another user?)

Did you try out the build that we use at TUM which comes from Bamboo?

Did you build it as root or as another user?

As a regular user on the host system.

Did you try out the build that we use at TUM which comes from Bamboo?

The WAR from the current develop combined with a local docker build with src/main/docker/Dockerfile is broken, too.

this is strange .. even with build container it does not work ..

@b-fein do you have an idea, why it does not build here: https://github.com/ls1intum/Artemis/actions/runs/3228188803/jobs/5283947520

@b-fein does this image work locally ?
The GH Docker image does not work for me

New finding:

Artifact of Github release 5.10.2 works
Bamboo artifact for the same commit broken

⇒ The issue seems to have been present before release 5.10.2, it just did not occur in Github Actions.

As follow-up we can think about changing the SecurityConfig similar to this https://github.com/kit-sdq/NewsList/blob/main/src/main/java/edu/kit/kastel/sdq/newslist/config/SecurityConfiguration.java But therefore, we might have to migrate some implementations.

Manually tested this with a docker-compose setup including a SAML test server and a mail server with an empty Artemis DB.
It took a while to process everything but that's probably my computer, but I get the SAML account registered in Artemis, GitLab and Jenkins and get logged in. So it seems like the SAML login is working with these changes!
After logging out I guess I'm keeping some kind of session cookie as after clicking SAML2 Login I'm logged in as the user I wanted to sign out before. Not sure if this is a bug or a feature. If I delete all cookies, I can log in again as a different user.
After playing a bit around, I also noticed that if I try going back (browser back button) from the external SAML page I eventually get to the page /accesdenied and if I then click the Artemis logo I get to the Artemis Login page, but I instantly get redirected to the external SAML page. So I can't leave this cycle without entering correct data in the external SAML page or waiting for a bit. It could of course be that this is a problem with my configuration! Also, probably not an issue introduced with these changes?!

That should not be related to this issue. Maybe we should raise another to improve the workflow :)

That should not be related to this issue. Maybe we should raise another to improve the workflow :)

As I'm not into the Spring Security Scheme I couldn't review the code and therefore just tested more thoroughly and already thought that this is not related but couldn't verify ;)
@b-fein I added an PR for the Java distro changes:

#5739

It should just be the old code (same as before the mentioned PR)


UI: Workflow list blocked when limit applied

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issue exists when I tested with :latest
[X] I have searched existing issues and could not find a match for this bug
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what did you expect to happen?
On the workflow list page, when the total number of workflows exceeds the limit applied in the query string:
(eg: total number of workflows = 150 and <base_url>/workflows/product-replacement-engine?&limit=140)
the application enters into a loop, continuously repeating the same calls, and blocking any other requests.

Version
v3.5.4
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
n/a

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}
n/a

Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded
n/a

Hmm this might be the same root cause as #12626. I can't tell from looking at the code why this would happen though and haven't had a chance yet to sit down and test it.

Seems the same. I believe the cause is in the dependency array of one of the useEffect

Yea that was what I checked first too, but I don't see anything obviously off with it (I even checked to make sure toString was stable).
If you're familiar with React, I refactored this whole component to hooks in #11891, where I actually noticed some existing useEffect calls had stale refs. I tested to make sure everything updated correctly too since it was a pretty sizeable and complex refactor (one of the largest I've done in recent years). I also reduced the number of network calls in that PR too (it was doing unnecessary lists).
One thing I did noticed was that I seemed to have written the wrong key for the offset query param, but you're not using offset, so I wouldn't think that would impact this specifically. Will fix that ofc, but that doesn't seem like root cause to me yet 🤔

I tested to make sure everything updated correctly too since it was a pretty sizeable and complex refactor (one of the largest I've done in recent years).

Yea I can't reproduce this locally, I only have one long-running eventsource request (SSE) if I don't change any filters:

Let me try building 3.5.4 specifically. #12626 suggests it might be an environment issue. Have you tried clearing your localStorage and similar caches?

Hmm this might be the same root cause as #12626.

Hmm actually your issue seems to be different. There none of the requests get cancelled, it just opens multiple (meaning the useEffect callback didn't happen) for some reason. In your case, they're quickly opened and closed in succession, ad infinitum.
Potentially same root cause, but the symptoms are different as of right now.

Let me try building 3.5.4 specifically

Huh I can't reproduce this on 3.5.4 locally either:

Ok I was able to partially reproduce this. With pagination on and when moving between pages, I get 2 SSEs and often neither will cancelled after moving pages (sometimes 1 will be cancelled).

The bizarre part is that there is no error in the console and I confirmed with a log statement and other observations that the useEffect callback and the listWatch.stop() therein is happening.
I can rewrite the logic so that it ensures there is only ever one listWatch, but in theory that shouldn't be necessary; I'm super confused why they're not being cancelled each time despite listWatch.stop() being called.
This also doesn't happen when I'm not interacting with the page, I still haven't been able to reproduce an infinite loop, nor one with no interaction required.

I can rewrite the logic so that it ensures there is only ever one listWatch with a ref, but in theory that shouldn't be necessary; I'm super confused why they're not being cancelled each time despite listWatch.stop() being called.

The plot thickens -- this doesn't fix it. Even with a singular ref, somehow multiple listWatches proliferate. Even when .stop() is called before any .start(). Even if I set the ref to null temporarily after a .stop()

With pagination on and when moving between pages, I get 2 SSEs

Found the main bug, the nextOffset of pagination is set by the ListWatch as well as a dep of its useEffect.
Removing that fixes the 2 SSEs. I'm not quite sure how you got an infinite loop though, as it should only happen twice since after the second one it should be constant 🤔
That also seems to mostly the the .stop() not working. Doesn't need a ref and works as-is for the most part.
It seems like .stop() may not work properly when called in quick succession after .start() as with the 2 SSEs. It seems to be latency dependent, so that would explain why only some people on certain connections have experienced it (e.g. in #12626 they are using HTTP/1). That one predates 3.5.x though and I'm not entirely sure how to fix it; it might be an RxJS issue or a Server or SSE issue with the quick succession.
So I'll fix the main problem that is a regression and hopefully we won't see the .stop() issue again nor infinite loops, but those may need further investigating.


[WIP] scoop services (first draft)

First draft PR for #1526
Services XML files will be located in ./bucket/services for the main bucket and ~/scoop/buckets/<bucketname>/services for other buckets.
Steps for testing
Checkout this PR and install sudo, winsw and php.
λ scoop install sudo winsw
λ scoop install php
λ scoop services install php
Installing a service will look for a suitable XML file in the services directory.
In this case, the service property of php is set to php-fpm.
It will copy <bucket>/services/php-fpm.xml to ~/scoop/persist/php/service/php-service.xml and create a copy of WinSW.exe at the same location (php-service.exe)
Start/stop/restart the php service with:
λ scoop services start php
λ scoop services stop php
λ scoop services restart php
Uninstall the php service with (php-service.xml and php-service.exe will remain untouched)
λ scoop services uninstall php
Lists all services that are available and their status (only show installed apps)
λ scoop services list
[C]onfigured
 | [I]nstalled
 |  | [S]atus
 |  |  |
[-][-][-] caddy (caddy)
[+][+][+] php (php-fpm)

All commands can be run with sudo.
Todo list:

[ ] show information after installing an app with a service
[ ] uninstall service before uninstalling an app
[ ] implement/test --all option
[ ] implement/test --global option
[ ] adjust schema.json
[ ] things I overlooked or haven't thought about jet ...

Services todo list:

[ ] apache
[x] caddy
[ ] minio
[ ] mongodb
[ ] nginx
[x] php
[x] php-nts
[ ] rethinkdb

@deevus thought about doing that at first. If you take a look at the sample-allOptions.xml you'll see that WinSW has many options and I believe it's unnecessary to reimplement them just for a service running with the least amount of configuration. If need the user can change all options afterward.

Sorry, I just got back from holiday. This is looking promising.
I share @deevus's sentiment that it would be very nice to do it in JSON and generate XML.
Nice because Scoop could act as a simpler wrapper around WinSW, making it easier for manifest authors to do what's needed most of the time, like executable, arguments, and environment. It would also be nice to generate defaults for the service id, name, description, logging and restart without having to put them in the manifest.
I get your points about WinSW supporting lots of options, and allowing users to change options later. Wouldn't it be possible to keep both these benefits if we generate the XML?

This is a nice improvement I find myself needing in scoop to ship many of the developer tools I work on. Is there someone working actively on it? :smile:

I just scoop install mysql'ed, but I'm not sure how to do service management. Apparently it's not yet possible with Scoop?


Page crashes after some time

Describe the bug
We have implemented ChartJs within a blazor page. When we go to the page everything works fine. Now if we leave the tab open for x minutes/hours, the website crashes and we get an error message.
I don't got this behaviour in Firefox yet, only in Chrome.
Which Blazor project type is your bug related to?

Server-Side

Which charts does this bug apply to?
 <Chart Config="_chartConfig" @ref="_chart" />
To Reproduce
Steps to reproduce the behavior:
We are using ChartJs v 2.0.2.
We implemented the chart in this way:
 <Chart Config="_chartConfig" @ref="_chart" />
InitChartData() is called in OnParametersSetAsync()
Now keep the page open. It can take up to a few few minutes or hours until the page crashes.
Additional context / logging
Add any additional context about the problem here.

Category: Microsoft.AspNetCore.Components.Server.Circuits.CircuitHost
EventId: 111
SpanId: 155a6ee05355f14b
TraceId: 7ba32bda0dc0df4e8fc029139ced18f9
ParentId:<PHONE_NUMBER>000000
RequestId: 800001cb-000a-f800-b63f-84710c7967bb
RequestPath: /_blazor
TransportConnectionId: MIPBcXk0NMVBF9fEKt-Z5w

Unhandled exception in circuit 'WPZDgZ09abqTNY91AXwwP7xZwcfpPAlEnyN229qjEuw'.

Exception:
Microsoft.JSInterop.JSException: Could not find a chart with the given id. 568bdcd0-6079-42a7-86bf-c5e66f502a2a
   at Microsoft.JSInterop.JSRuntime.InvokeAsync[TValue](Int64 targetInstanceId, String identifier, Object[] args)
   at ChartJs.Blazor.Chart.OnAfterRenderAsync(Boolean firstRender)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.GetErrorHandledTask(Task taskToHandle)

Code example
Please provide full code examples below where possible to make it easier for the developers to check your issues.
Please also add the working JavaScript equivalent so we can see what you were trying to achieve.
protected override async Task OnParametersSetAsync()
{
	if (CustomerId > 0)
	{
		_customer = await Kunde.GetKundeAsync(Kundennummer.ToString());
		if (_customer is not null)
		{
			/* More Stuff is happening here */

			Umsatzdaten = new Dictionary<int, Jahresumsatz>();
			for (int i = 0; i < UmsatzJahre; i++)
			{
				if(!Umsatzdaten.ContainsKey(i))
				{
					Umsatzdaten.Add(i, await customerService.GetCustomerDataAsync(_customer.KUND_A_NR, DateTime.Now.Year - i));
				}
			}

			InitChartData();
		}
		else
		{
			/* More Stuff is happening here */
		}
	}
}

private void InitChartData()
{

	_chartConfig = new LineConfig
	{
		Options = new LineOptions
		{
			Responsive = true,
			Title = new OptionsTitle
			{
				Display = true,
				Text = $"Umsatz der letzten {UmsatzJahre} Jahre"

			}
		}
	};

	_chartConfig.Data.Labels.Clear();
	_chartConfig.Data.Datasets.Clear();

	for (int i = 1; i <= 12; i++)
	{
		_chartConfig.Data.Labels.Add(System.Globalization.CultureInfo.CurrentCulture.DateTimeFormat.GetMonthName(i));
	}

	for (int i = 0; i < UmsatzJahre; i++)
	{
		string lbl = (DateTime.Now.Year - i).ToString();
		if (Umsatzdaten.ContainsKey(i))
		{
			IDataset<decimal> tmpDs = new LineDataset<decimal>(Umsatzdaten[i].ToIEnumerable())
			{
				Label = lbl,
				BackgroundColor = ColorUtil.FromDrawingColor(ChartHelper.GetColor(i)),
				BorderColor = ColorUtil.FromDrawingColor(ChartHelper.GetColor(i)),
				Fill = FillingMode.Disabled,
			};
			_chartConfig.Data.Datasets.Add(tmpDs);
		}
	}
}

I have the same issue on a page that refreshes the displayed data every 10 seconds.
I just had a quick look at the base chart class and it appears to use an ID from the Config for the chart component. Since the Config is being recreated, a new ID is being generated. This means that Blazor cannot locate the original chart component to check for changes in the DOM.
Workaround: Do not recreate the Config instance within the InitChartData() method. Create an instance in the Page Initialize() method, and then update the properties, such as Options. You will need to keep the original Config instance so the ID doesn't change.
Possible Fix: Allow the ID to be specified in the Config constructor, or make the ID property read/write.
Note that this means that all sample code where this technique is being used will need to be changed as well.


[Update Request]: Termius 8.8.3 -> 8.9.7

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Termius.Termius
Package Version
8.9.7
Please describe the changes you would like to see
It's been couple weeks that the winget installation of Termius is not succeeding to "hash mismatch" This is because there is a new version.
As of 5 min ago 8.9.7
Source : https://autoupdate.termius.com/windows/Install Termius.exe
(Same source used in Winget)
PackageVersion: 8.9.7
InstallerSha256: B159761B935B8900DA7DD255B75788C23F435E9F484E4FC38856EDD4BAB3FAEA

Hello, if this is a frequent pain point for you, then we now have a service that fixes the "hash mismatch" problem once and for all: https://winget.pro/pkgs-winget-installer-hash-does-not-match/

For future ref since this uses a vanity URL, it's going to run into issues quite often where the hash of the installer wont match. This can be solved for by installing via the ms store version: winget install 9nk1gdvpx09v

https://autoupdate.termius.com/windows/latest.yml


Support looking up types by (approximate) name instead of just position

Through discussion here I learned that the protocol doesn't support looking up things by name. Previously using tagging programs it was quite easy for me to fuzzy search for a class name in my code base. Now if I want to look at the definition of something even if I know its exact name I have to insert a reference to it into my current source file in order to be able to look it up. Ideally you could query a name and get back a list of possibilities, enabling editors to have features like narrowing down what class you mean as you type.

You can query a symbol by name using workspace/symbol. Clients should perform a "Go to Definition" when selecting the result.

You can query a symbol by name using workspace/symbol. Clients should perform a "Go to Definition" when selecting the result.

I see they changed the protocol to say "Clients may send an empty  string here to request all symbols.".  Well, that's something.
And there's a "partial result" thing now.  Got to look into that.


Fix/vite build issues

GitHub Issue
I fixed everything - check this out for more details
Changes
A summary of the changes in the code in this PR. If making any UI changes consider adding screenshots or gifs.
Packages Added
Brief list of any packages added, and if folks need to rerun yarn install to run locally
Checks
Before making your PR, please check the following:

[ ] Critical lint errors are resolved
[ ] App runs locally
[ ] App builds locally (run the build command for any impacted package and check for any errors before the PR)

interesting @skuhlmann - @0xRowdy what do you think about sams fix?

closing for now


Migrate Physical Location test from E2E to integrate test suite TP

What does this PR (Pull Request) do?

[x] This PR fixes  is not related to any Issue

Which Traffic Control components are affected by this PR?

CDN in a Box
Traffic Portal
CI tests

What is the best way to verify this PR?

Make sure all tests are migrate in integration/Data/physlocations.ts directory.
https://github.com/apache/trafficcontrol/blob/master/traffic_portal/test/e2e_deprecated/physLocations/phys-locations-spec.js
Go to traffic_portal/test/integration and run
npm test -- --params.baseUrl https://localhost --capabilities.shardTestFiles true and make sure test all pass.
Make sure CI tests pass also.
If this is a bug fix, what versions of Traffic Control are affected?

The following criteria are ALL met by this PR

[ ] This PR includes tests OR I have explained why tests are unnecessary
[ ] This PR includes documentation OR I have explained why documentation is unnecessary
[ ] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[ ] This PR includes any and all required license headers
[ ] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

Incorrect expectation here: https://github.com/apache/trafficcontrol/blob/85e799993e78dadd3fd599815f15f597a6337a15/traffic_portal/test/integration/specs/PhysLocations.spec.ts#L47
Should be physical locations instead of parameters


Allow new domains to be absolute

Allow the new domain name to be input absolute (with a dot at the end). To keep the rest of the logic working as-is, remove it fairly early in the function.
Would have loved to use str.removesuffix() but that's python v3.9+ only.

Hi,
Thanks for this PR. In my local code I use rstrip('.') (but this is fine as well)


adding link to service principal doc from kubernetes trouble shooting…

… section

This change is

@JackQuincy,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Issue with createDeveloperProduct.

Upon using createDeveloperProduct, I get this error shown:
BloxyHttpError [BloxyHttpInvalidStatusCodeError]:

Invalid status code in response. Body: {"errors":[{"code":0,"message":"No HTTP resource was found that matches the request URI 'http://develop.roblox.com/v1/universes/2095224927/developerproducts'."}]} | Status code: 404, status: undefined. Possible reasons:
- Invalid request method (i.e. POST instead of GET)
- Resource removed / updated to another route
- You're not logged in (use .login on the client to log in)

    at validStatusCode (C:\Users\Spiral\Desktop\bot\node_modules\bloxy\dist\controllers\rest\response\handlers\validStatusCode.js:30:29)
    at C:\Users\Spiral\Desktop\bot\node_modules\bloxy\dist\controllers\rest\response\RESTResponse.js:12:78
    at Array.map (<anonymous>)
    at RESTResponse.process (C:\Users\Spiral\Desktop\bot\node_modules\bloxy\dist\controllers\rest\response\RESTResponse.js:12:63)
    at RESTRequest.<anonymous> (C:\Users\Spiral\Desktop\bot\node_modules\bloxy\dist\controllers\rest\request\RESTRequest.js:26:29)
    at Generator.next (<anonymous>)
    at fulfilled (C:\Users\Spiral\Desktop\bot\node_modules\tslib\tslib.js:111:62)
    at processTicksAndRejections (internal/process/task_queues.js:97:5) {
  statusCode: 404,
  status: undefined
}

I believe that this is the doing of Bloxy itself and not my code. I will not post my code publicly so if you feel you need to review my code for further investigation, contact me at SpiralRBX#5787

It would be helpful to see the code you used to see if there was a possible hiccup in your code

It would be helpful to see the code you used to see if there was a possible hiccup

Okay. I will post the code when I am able to get on my pc.

It would be helpful to see the code you used to see if there was a possible hiccup

`const bloxy = require("bloxy");
const client = new bloxy.Client({
credentials: {
cookie: ""
}
});
function wait(ms) {
return new Promise((resolve) => setTimeout(resolve, ms))
}
async function run() {
const authenticatedUser = await client.login();
console.log(Logged in as ${authenticatedUser.id})
}
async function exec(price) {
await wait(200)
console.log("finished exec " + price)
let addPromise = client.apis.developAPI.createDeveloperProduct({
universeId:<PHONE_NUMBER>,
name: "Test" + price,
description: "Product",
priceInRobux: price,
iconImageAssetId:<PHONE_NUMBER>
}).catch((res) => {
console.error(res)
})
await addPromise
}
// Instead of just calling run, wait for it to finish or you might run into issues of not being logged in
run().then(async () => {
for (let step = 1; step < 10000; step++) {
await wait(3000)
await exec(step) // Added await here to wait for product to be created
console.log(Product Named "${step}" created!)
}
})`

Just make a similar reproducible version of your code

Just make a similar reproducible version of your code

Alrighty!
const bloxy = require("bloxy");
const client = new bloxy.Client({
  credentials: {
    cookie: ""
  }
});
function wait(ms) {
  return new Promise((resolve) => setTimeout(resolve, ms))
}

async function run() {
  const authenticatedUser = await client.login();
  console.log(`Logged in as ${authenticatedUser.id}`)
}

async function exec(price) {
  await wait(200)
  console.log("finished exec " + price)
  let addPromise = client.apis.developAPI.createDeveloperProduct({
    universeId:<PHONE_NUMBER>,
    name: "Test" + price,
    description: "Product",
    priceInRobux: price,
    iconImageAssetId:<PHONE_NUMBER>
  }).catch((res) => {
    console.error(res)
  })

  await addPromise
}

// Instead of just calling run, wait for it to finish or you might run into issues of not being logged in
run().then(async () => {
  for (let step = 1; step < 10000; step++) {
    await wait(3000)
    await exec(step) // Added await here to wait for product to be created
    console.log(`Product Named "${step}" created!`)
  }
})

I've found the issue, it was that Roblox are using query parameters to create a developer product, while bloxy uploads a JSON body.
I'm implementing a fix for it now!


Not available in integrations anymore?

Hi,
Since I've upgraded to 1.4.0, the integration doesn't show up in the new integrations modal anymore. When I downgraded to 1.3.0 it worked again.
Maybe something worth looking at.
I will use 1.3.0 for now.

I cannot reproduce the error
I have done the following:

deleted the integration completely
installed version 1.3
update to 1.4.

Integration is visible.
Try this:
Go to "Device and Service" > "ADD INTEGRATION" search for SAJ H2

I think it indeed had something to do with my setup. It seems to be updated to 1.4.0 automatically (even though hacs doesn't do that as far as I know) and now it works as it should.
Sorry for the unnecessary issue and have a nice day!


[go_router] access matched location with the actual path parameter

Use case
In this route the matchedLocation will return album/:albumId. Is there a builtin way of getting the route prefix and the actual value of the path param? If the album id is 3 I want to get album/3. Does this exist?
              GoRoute(
                path: 'album/:albumId',
                builder: (BuildContext context, GoRouterState state) {
                  print(state.matchedLocation);
                  return AlbumScreen(
                    albumId: state.pathParameters['albumId'],
                  );
                },
                )

Proposal
I want to be able to get the matched location with the actual value of the path param.

@momrak
Can you check this issue and see if it helps in your case or not ?

Thanks for the feedback.

There is a private method you can copy over to your code base as a workaround
https://github.com/flutter/packages/blob/adf7adf983fc8a56adeaf2b84e580698acdb9638/packages/go_router/lib/src/path_utils.dart#L73
If we were to do this, I think it will be to open up this file in go_router

Nice, thank you 👍

I think there is an issue as @momrak  mentioned to be addressed
https://github.com/flutter/flutter/issues/134402#issue-1889775023
The albumId param value is not the value returned but rather the parameter name  :albumId  when trying to access it by  state.pathParameters['albumId']

I think there is an issue as @momrak mentioned to be addressed #134402 (comment)
The albumId param value is not returned but rather the parameter name :albumId  when trying to access it by state.pathParameters['albumId']

Same Issue at my side,
Caller Side:
GoRouter.of(context).goNamed(                                 Pages().manageCourses.name,                                 extra: course,                                 pathParameters: {"courseId": course.ref.id});
Reciever Side:

oops this looks worse than i thought thanks for pointing out

I can't reproduce the issue this is the code I am using

import 'package:flutter/material.dart';
import 'package:go_router/go_router.dart';

/// This sample app demonstrates how to use GoRoute.onExit.
void main() => runApp(const MyApp());

/// The route configuration.
final GoRouter _router = GoRouter(
  routes: <RouteBase>[
    GoRoute(
      path: '/',
      builder: (BuildContext context, GoRouterState state) {
        return const HomeScreen();
      },
      routes: <RouteBase>[
        GoRoute(
          path: 'album/:albumId',
          builder: (BuildContext context, GoRouterState state) {
            print(state.matchedLocation);
            print("state.pathParameters['albumId'] is ${state.pathParameters['albumId']}");
            print("state.pathParameters['albumId'] from context is ${GoRouterState.of(context).pathParameters['albumId']}");
            return Text(state.pathParameters['albumId'] ?? " null");
          },
        )
      ],
    ),
  ],
);

/// The main app.
class MyApp extends StatelessWidget {
  /// Constructs a [MyApp]
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp.router(
      routerConfig: _router,
    );
  }
}

/// The home screen
class HomeScreen extends StatelessWidget {
  /// Constructs a [HomeScreen]
  const HomeScreen({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Home Screen')),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            ElevatedButton(
              onPressed: () => context.go('/album/123'),
              child: const Text('Go to the Details screen'),
            ),
          ],
        ),
      ),
    );
  }
}

Can You also try with Shell Routes. ? I am experiencing issue for routes in shellRoutes.

@iharoonamjad Can you provide a repro?

I am also unable to reproduce, could be problem within my code.
Will share solution for my issue if caught.

I can't reproduce the issue this is the code I am using
import 'package:flutter/material.dart';
import 'package:go_router/go_router.dart';

/// This sample app demonstrates how to use GoRoute.onExit.
void main() => runApp(const MyApp());

/// The route configuration.
final GoRouter _router = GoRouter(
  routes: <RouteBase>[
    GoRoute(
      path: '/',
      builder: (BuildContext context, GoRouterState state) {
        return const HomeScreen();
      },
      routes: <RouteBase>[
        GoRoute(
          path: 'album/:albumId',
          builder: (BuildContext context, GoRouterState state) {
            print(state.matchedLocation);
            print("state.pathParameters['albumId'] is ${state.pathParameters['albumId']}");
            print("state.pathParameters['albumId'] from context is ${GoRouterState.of(context).pathParameters['albumId']}");
            return Text(state.pathParameters['albumId'] ?? " null");
          },
        )
      ],
    ),
  ],
);

/// The main app.
class MyApp extends StatelessWidget {
  /// Constructs a [MyApp]
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp.router(
      routerConfig: _router,
    );
  }
}

/// The home screen
class HomeScreen extends StatelessWidget {
  /// Constructs a [HomeScreen]
  const HomeScreen({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Home Screen')),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            ElevatedButton(
              onPressed: () => context.go('/album/123'),
              child: const Text('Go to the Details screen'),
            ),
          ],
        ),
      ),
    );
  }
}

@chunhtai would try and test this album/:albumId as a subroute of /.

would you try and test this album/:albumId as a subroute of /.

Isn't this what the code does already?


Silence /healthz requests in log

Django logs all HTTP transactions by default, but ideally GET requests on /health-check would be silenced or retargeted for DEBUG level only. Otherwise that constant traffic will crowd out useful log info.

This is a log message from gunicorn itself and I haven't found a way to silence it by hooking into the filtering system.
We'd be best served by putting nginx in front of gunicorn and using that access log which allows you to silence logging per specific routes but even then I don't know if you can have it silence only 200 but let the errors through. Would be our best option overall, beyond living with the the messages.
Or ... we could funnel this into a different syslog output and not stdout

alternatively, we can change httpGet to an exec/command livenessProbe that calls manage.py healthchecks

It's true--this is logged by WSGI itself with the equivalent of a print() statement, so to silence it we would need to write custom middleware for that level. Messing with logging is fruitless for this case. There are a couple examples of such hackery on the net. We could key off the DEBUG env var to decide whether or not to log.

I was dropping all the way down to doing a extended gunicorn.glogging to see if that panned out. Even middleware won't be enough I think to exclude it but I may be wrong


Update terminal-notifier to 2.0.0

Under the new OS X system, it seems that the old version of the terminal-notifier does not support the Link field. I have seen other people feedback similar questions. #13
I updated the terminal-notifier in the code to their latest version 2.0.0. The binary file use zip and gz compressed. The local test is ok.

Thanks for doing this...I've never had anyone update the binary for this repo so I need to check a few things before I merge.  I'll keep you posted.


Log stream frames in long header packets

0-RTT stream frames in long header packets are neither output in debug logging nor in qlog.

Fixed by #3922.


No reaction after click on Record/stop button

Hi, there seems to be another problem:
Sometimes when I click the record/stop button, Captura 7.01 (Windows 10) does not seem to react.
Still, when I want to close the program "by force" via right click on the task in the Windows task bar, Captura asks me whether I really want to close the program - even though a recording is running. (I do not get any other signal that a recording is running, though.)
And when I click on the record/stop button again, there is again no reaction - and no file is saved.
Kind regards, Mat

What are your system's CPU and RAM specifications?

Intel Pentium CPU N3710 @ 1.60 GHz,
4GB RAM

This problem is happening due to higher CPU requirement of Captura. I have not found the exact minimum but almost 2 GHz might be required.

Ok, thanks.

Added Recommended System Requirements in README and FAQ


Complete force+cancel

Functionality has been prepared but not completed in the backend,

Enable cancel/force in frontend by setting cancellable/forcible on RunlogLine dto
Set on dto from RunLogLine protocol model
Set on RunLogLine protocol model by ?
Handle force/cancel requests in ?

Note #465 which is related in regards to alarm instance

Looking into progress reporting as well

Test programs
Watch: Block Time > 3s
    Info: W

Info: A
Wait: 3s
Info: B


IsCompleted is checked instead of the metadata in AnalyzeIamPolicyLongrunning.

IsCompleted is checked For AnalyzeIamPolicyLongrunning API instead of the metadata since this is what customers would first care about.

Ah, we crossed PRs. See #1454 .
I'll close this one as a duplicate basically. Thanks!


Notification stack

Description
Create the notification stack component as shown in the video attached below.
Animation Preview

Screenshot/Video: https://github.com/user-attachments/assets/e7fc2979-dc60-4870-b603-fdd6b4ca9360

[!NOTE]
Please refer to the Animata contributing guidelines available here for rules on how to contribute to the project. Let us know in the comments if you’re working on this issue, and feel free to ask any questions!

Requirements

[ ] Create a new animated component that matches the provided design.
[ ] Add example usage of the component in the documentation or storybook.

 Guidelines & Best Practices

Write reusable and clean code following the project’s style guide.
The component should be fully customizable. The user should be able to modify props such as duration, easing, color, and size where applicable.
Follow the Contributing Guidelines before submitting your pull request.

[!IMPORTANT]
To ensure more contributors have the opportunity to participate, we kindly request that:

Each contributor submits only one pull request related to this issue.
If you're interested in working on this issue, first check if it has already been claimed. If not, assign yourself to the issue and move it to In Progress on the project board.

Additional resources
Posted on X by: https://x.com/laurentdelrey/status/1702369534823301207

Checklist

[ ] I have read and followed the contributing guidelines.
[ ] I have developed the component according to the requirements.
[ ] I have tested the component across different browsers and screen sizes.
[ ] I have documented the component for future reference.
[ ] I have submitted a pull request with appropriate labels and referenced this issue number.

@sudhashrestha Hello Mam, Can you plz assign this to me, I want to work on it.

Hi @pranjalee30
I hope you're doing well! Since it's been a while since this issue was assigned, I wanted to check in and see if you're still planning to work on it. If so, could you kindly share an update by tomorrow?
If I don't hear back by then, I’ll unassign the issue to allow others to take it up.
Thanks for your understanding!


fix(client): Don't show $ws when not used WebSockets

In client, IntelliSense always shows (route).$ws property.
I think client doesn't have to show (route).$ws property when route doesn't use WebSocket helper.
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

Hi @nakasyou !
Super cool! I love it.
I want to merge this, but do you have any idea how to write tests? It's better to write a test, though it seems a little bit difficult because it is Types of matter.

@yusukebe

I want to merge this, but do you have any idea how to write tests? It's better to write a test, though it seems a little bit difficult because it is Types of matter.

I searched about it, then I found that Vitest has a way of testing types. So I made test using this.
Reference:

https://www.totaltypescript.com/how-to-test-your-types
https://vitest.dev/guide/testing-types.html

@nakasyou
Awesome! That's best.
Let's ship it. Thanks!


Remove 'attempt to delete non-existent session' warning from destroy()

This PR removes a log warning in destroy(), which fired whenever there was an attempt to delete a non-existent session.
This log warning has been alarming for users of the passport library (see issue #91); where login results in a call to req.session.regenerate(…), which in turn calls calls destroy() and then generate().

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Uninstalling NVM Completely

Hello!
I've installed NVM with the script provided:
curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.33.2/install.sh | bash
If I am understanding everything correctly the only thing this file seems to have done is added 3 lines to the very bottom of my .bashrc and created a folder called .nvm in my ~ folder.
In this same ~ folder I have a handful of my development sites (in a separate folder) do I need to do anything special to completely remove nvm or will removing the ~/.nvm and the bottom 3 lines of .bashrc be enough?
Does the install.sh script make any other modifications to files elsewhere after that specific command (IE do I have to worry about anything being modified in my ~/apps folder where my sites are hosted).
Thanks!

(duplicate of #298)
No, all you need to do is remove $NVM_DIR (~/.nvm is the default), and the relevant 2 lines in your profile file (but you also can leave those behind, since they're guarded by a file existence check).

Great, thanks! - No files outside of the $NVM_DIR are modified at all? Seems I did that right then.

@zackeryfretty one of your profile files will be modified by the install script, but nothing else should be.

Is that really all you have to do? I've removed the $NVM_DIR, removed all nvm related lines in my bash profile, and when I type nvm in my terminal, I still get all the options associated with nvm. So it looks like it is still installed, even if all the versions of node are uninstalled...

your .npmrc file may has a prefix set

If so, nvm will have made you remove it, but nvm won’t leave one behind. This is good anyways, because the prefix setting is best avoided in general.

Interesting.
I followed the uninstall instructions in the readme here.
Then I tried out n, but recognized that npm globally installed packages still landed in ~/.nvm/*.
So I checked the .npmrc file and there was still that prefix set.
I don't know why it was still there.

Presumably you put it there in the first place :-) nvm certainly didn’t.

Yeah, very likely, I use npm since years, who knows which workaround ended up there^^
Anyway, nvm is a really great tool!


[Documentation Issue] : Documentation Changed

Brief Description 📝
There we have to change somethings on the Readme
Select the open-source program you are contributing to
Other
Code of Conduct

[X] I agree to the CONTRIBUTING GUIDELINES of this project.

Hey @shuvadeepmondal👋! Thanks for raising this issue and demonstrating your interest in contributing to our project!🚀
We value your engagement and eagerly anticipate your contributions. Should you wish to get involved, please use the /assignme command to self-assign the issue.
This issue provides an excellent opportunity for us to enhance our project, and we're excited to have you join us! Please take a moment to review our CONTRIBUTING GUIDELINES before getting started.
If you have any questions or need assistance, feel free to reach out to us on LinkedIn.❤

/assignme


Move to Debian 12 and fix Uptime Kuma

Description

Update all images that can to debian 12
fix the Uptime Kuma build

This PR is sort of linked to https://github.com/parkervcp/eggs/pull/2844
All Submissions:

[x] Have you ensured there aren't other open Pull Requests for the same update or change?
[x] Have you created a new branch for your changes and PR from that branch and not from your master branch?

@QuintenQVD0 These all build correct?

Some will fail the first time as they depent on a image also build here. The rest should yes. I normaly checked if the packages still exits. But i have not yet test builded them.


[Bug]: Can't connect to the hosted app, runtime error >24h

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Describe the bug and reproduction steps
I was testing the app 2 days ago when I faced an issue (#4857). Issue was solved yesterday with the new release, but I'm getting the attached error ever since, and it's been approximately 48h from the last time I used the app (Robert pointed out timeouts are deleted every 24h).
OpenHands Installation
app.all-hands.dev
OpenHands Version
No response
Operating System
None
Logs, Errors, Screenshots, and Additional Context

Hi @Belzedar94. Are you hoping to recover your session with all the files? Or do you want to just use the app and it's stuck like that?

Hi @Belzedar94. Are you hoping to recover your session with all the files? Or do you want to just use the app and it's stuck like that?

Hey buddy! I was hoping to recover my session and files, but if there is no way and creating a new project fixes that, I'll do it!

I've asked the group but there is a high chance that you have to exit the project and restart....

Yes just confirmed you have to restart a project.. sorry about this. They are thinking about making it more forgiving in the future.
Make sure you download and save your work periodically. Or better yet push it to a branch via OpenHands :)

It happened again today, but for a session that was 11h old. After getting the attached runtime error for hours, it suddenly stopped popping-up, but then the client got indefinitely stuck in "Waiting for client to become ready....". Refreshing didn't solve it

This should be much better now with the changes in 0.15.0. Will leave it open but please let us know if you run into it again < 24h.

Hi @Belzedar94 , if you're still encountering this please tell us. But I'll close this issue for now!


Update update_ddns_dnsmadeeasy.py

On August 1st 2015 DNS Made Easy will be changing the url for update calls to http://cp.dnsmadeeasy.com/servlet/updateip?
In addition the url to grab the IP will change to http://myip.dnsmadeeasy.com
Here is a link to the new documentation: http://www.dnsmadeeasy.com/dynamic-dns/

Pretty please merge these :-)  Just lost half an hour to this issue and this fixes it perfectly

Sorry, this had slipped from my radar. Merged now. :-)


Crash Report

name = "rainfrog"
operating_system = "Mac OS 14.6.1 [64-bit]"
crate_version = "0.1.12"
explanation = """
Panic occurred in file 'src/database.rs' at line 140
"""
cause = "index out of bounds: the len is 0 but the index is 0"
method = "Panic"
backtrace = """
0: 0x1012cb600 - core::panicking::panic_fmt::he22466887efdf1d5
1: 0x1012cb77c - core::panicking::panic_bounds_check::h2e36cc48ec80c99e
2: 0x100c98a64 - rainfrog::database::should_use_tx::h63e553b4256700e4
3: 0x100c090c0 - rainfrog::app::App::run::{{closure}}::{{closure}}::ha0478c6505e5f947
4: 0x100bfc420 - tokio::runtime::task::core::Core<T,S>::poll::h7419f0cf72c65589
5: 0x100c778d4 - tokio::runtime::task::harness::Harness<T,S>::poll::h187471fc6c6570e0
6: 0x100f34074 - tokio::runtime::scheduler::multi_thread::worker::Context::run_task::h77437fd625c9d847
7: 0x100f32f98 - tokio::runtime::scheduler::multi_thread::worker::Context::run::hfa4151b07d0658ce
8: 0x100f36d10 - tokio::runtime::context::runtime::enter_runtime::h52492ae13d985528
9: 0x100f32afc - tokio::runtime::scheduler::multi_thread::worker::run::h97429d61333adc17
10: 0x100f424ec - tokio::runtime::task::core::Core<T,S>::poll::h7a5795830bf541d1
11: 0x100f226a8 - tokio::runtime::task::harness::Harness<T,S>::poll::h610f65f1ff84c0a3
12: 0x100f25adc - tokio::runtime::blocking::pool::Inner::run::hc623579bfa5eb581
13: 0x100f42e04 - std::sys::backtrace::__rust_begin_short_backtrace::h42aff85751f1f775
14: 0x100f3b820 - core::ops::function::FnOnce::call_once{{vtable.shim}}::h85d2cb21bdaeb412
15: 0x101282fdc - std::sys::pal::unix::thread::Thread::new::thread_start::h1464447dee17c73c
16: 0x1930d1f94 - __pthread_joiner_wake"""

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Is this better here, or in an email?

thanks @tjweir, github issues is good. did this happen on startup or after running a specific query?

Yes, I had something like this in the query editor.
-- select * from table1;
select * from table1 where x = 1;

ah it's quite similar to https://github.com/achristmascarl/rainfrog/issues/32, because of the semi-colon in the comment. i'll fix it in the next release, in the meantime you'll need to remove comments from the top of the query 🙏

"in the meantime you'll need to remove comments from the top of the query"
Don't boss me around!  :)
Sounds good, shall I close in favour of #32?

Sounds good, shall I close in favour of #32?

yep that works! thanks

Closed as dupe of #32


[google_sign_in]Added GoogleHttpClient to google_sign_in

Adding support for GoogleHttpClient to the google_sign_in package in Flutter

@cyanglaz any updates here?

@mehmetf minor version change right?

@mehmetf looks good now?

Thanks! Now the PR looks in good shape. However, I want to understand the rationale. Is this unblocking something you were not able to do before? or is it for convenience?
My concern is that the HttpClient has nothing much to do with the signin logic and adding package:http to the dependencies of every single client using this is not good.

This essentially allows a developer to enable three-legged OAuth by enabling application calls to Google APIs on behalf of end users, for example to allow the app to access a user's BigTable data via a consent screen.
It is an implementation of Google's Java HTTP Client Library in Flutter.

Thanks for the explanation. The motivation makes sense however could you not import a package that does this?
Retrieving a "httpClient" for an account looks odd to me. What if, for instance, my application has its own custom httpclient that adds other headers?
One solution could be: Does httpClient support interceptors that let you append headers? It would be nice for the account to provide an interceptor that can be installed and composed.

Hmm, that makes sense, although this would serve to only initialize the Google API's in this way -
Source 1
DialogflowApi(Client client, { String rootUrl: "https://dialogflow.googleapis.com/", String servicePath: "" })
Source 2
BigqueryApi(Client client, { String rootUrl: "https://www.googleapis.com/", String servicePath: "bigquery/v2/" })
Allowing the user to instantiate top-level variables in the main.dart file, and pass around the authorized client to the entire application and thereby "authorizing" once.
The interceptors seem like a valuable addition though! However I would need some assistance implementing them - I'm guessing constructors would do the trick?

Yeah absolutely. I have no doubt having a GoogleHttpClient that integrates well with GoogleSignIn is valuable. I am simply questioning whether it should be part of the plugin or not. You can attempt to create this as a separate package and see whether it can be done. We should be able to build Dart packages on top of plugins. You can create an HttpClient that expects a User in a named factory and generates an authenticated instance. All without touching the plugin code.
Interceptors are nice but that would require a bigger change as you point out.

@rishab2113 Any updates on this? Do you have a strong opinion on having this in the google_sign_in plugin or you also think it is better to have a separate package to do so.

Personally, I think having it would benefit a lot of users, however, personally I’ve found a non-Flutter solution to this.
I feel like having a way to automatically generate httpclients based on a user login is a valuable tool, and should be part of  the sign in package!

@rishab2113 Thank you for providing this PR. Unfortunately, after discussion with @mehmetf, we think this feature can be in a different package and we prefer not to merge this at this point. I am closing this PR now.


Added no_proxy to default docker template

The template for /etc/default/docker was missing the option for rendering the no_proxy environment variable.  I've added the option after the http_proxy and https_proxy options.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@yatsura

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.


Move AppPool doc content into Publish IIS doc

Fixes #2532

Hi @GuardRex, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@GuardRex  is this ready for review and merge?

@Rick-Anderson Yes --- ready for a look see :eyes:


Claim format independence

The introduction states "This specification is designed to be both Claim format and transport envelope agnostic"
This statement is not correct because the claims have to be formatted in JSON.
Do we want to keep claim format independence, or should we limit the specification to only apply to the W3C VC data model? This restriction will simplify the specification in a number of ways
a) format is no longer needed, because regardless of the crypto format the credential and presentation are always the same format so the same input descriptor can be used for all different crypto formats.
b) path can be simplified to be a single element rather than a set in most cases.

How would that work with JWTs? Note the wording in the first sentence of the encoding section below. I guess it can be handled during decoding and apply the JSON path on the resulting VC.
From https://www.w3.org/TR/vc-data-model/#jwt-encoding
`
For backward compatibility with JWT processors, the following registered JWT claim names MUST be used instead of, or in addition to, their respective standard verifiable credential counterparts:
exp MUST represent the expirationDate property, encoded as a UNIX timestamp (NumericDate).
iss MUST represent the issuer property of a verifiable credential or the holder property of a verifiable presentation.
nbf MUST represent issuanceDate, encoded as a UNIX timestamp (NumericDate).
jti MUST represent the id property of the verifiable credential or verifiable presentation.
sub MUST represent the id property contained in the verifiable credential subject.
aud MUST represent (i.e., identify) the intended audience of the verifiable presentation (i.e., the verifier intended by the presenting holder to receive and verify the verifiable presentation).`

On the last call I promised a magical sentence that would make all our single-JSONPath-selector dreams come true, so here goes - assume this is the very first line of text in the Input Evaluation section:

The inputs received MAY be encoded in a data model that requires processing or transformation steps to produce the bytes Presentation Exchange Input Evaluation intends to target and evaluate. You MUST process or transform the items received in accordance with their data model prior to evaluation, and proceed with the steps listed below using the output of whatever the data model's processing and transformation steps produce. For example: if an item is a JWT variant of a W3C Verifiable Credential, one would be required to execute the processing steps required by the W3C Verifiable Credential specification to yield the .vc subtree from the item, which the implementer would then use for processing in the steps below.

instead of, or in addition to

the WG compromise that launched a thousand non-interoperable implementations...
Here's my attempt at making this more explicit, and hopefully not too opinionated:
The inputs received MAY be encoded in a proofed representation of the VC data model that requires processing or transformation steps to produce the bytes Presentation Exchange Input Evaluation ~intends~expects to to target and evaluate**, namely, what the VC data model defines as a "credential", more colloquially a "payload" in JSON**. You MUST process or transform the items received in accordance with their representation and proofing method as defined in the VC data model prior to evaluation, and proceed with the steps listed below using the output of whatever the data model's processing and transformation steps produce. For example: if an item is a JWT variant of a W3C Verifiable Credential, one would be required to execute the processing steps required by the W3C Verifiable Credential specification to yield the .vc subtree from the item, which the implementer would then use for processing in the steps below.

@bumblefudge Well done!


Don't listen on docker interfaces

https://community.home-assistant.io/t/realtime-camera-streaming-without-any-delay-webrtc/258216/419

https://github.com/AlexxIT/go2rtc/blob/38a18cab6226db6f8d1982a425da6a4de58bed7e/pkg/webrtc/api.go#L30-L32


Maximum packet size

Hi, I am trying to publish data of around 5KB size. I changed the MQTT_MAX_PACKET_SIZE to 5000, but it doesn't seem to work.
The maximum I have been able to send is 1342 bytes. It seems like an odd number of bytes for maximum, but it is what I have observed. Anything more than that and the code just gets stuck.
Edit: The code gets either stuck or it just returns publish failed when I publish it to the topic.
Is there a way to send more than 1342 bytes in a single packet/message?
I am sending messages to AWS broker which needs to have 5 KB packets, anything less than that is inefficient since I have to pay for 5KB message while sending much less than that.
Any help is appreciated.

What device are you running that on? It comes straight out of the available ram, so a arduino like device is going to struggle reserving that much memory.

I am running on Arduino M0, 32KB RAM. I don't think that is an issue because I ran a freeRAM function and it said I had about 8-10 KB of RAM still available.

First of all - To me it seems like a wrong choice of service (or service provider) if you need to inflate packet size to some specific amount of KB.
You may switch to some other and cheap Arduino compatible hardware and check if the problem persists there as well (options could be ESP8266 or ESP32 - Wiki(https://en.wikipedia.org/wiki/ESP8266) etc.). Or you may switch to some other Arduino compatible MQTT library and give it a try.

The constraints are that I cannot change the hardware. I am using ATWINC1500 wifi module and arduino M0/SAMD21.
Is the packet size related to the hardware, have you observed before that for ATWINC1500, the packet size is less than for ESP8266 or ESP32?
I need to make it work for AWS, since they dropped the pricing and it is by far the most economical among all like IBM/Azure etc. for my application.


download dir doesn't follow xdg

https://wiki.archlinux.org/title/XDG_user_directories
my download_dir was set to /home/user/Videos/YouTube instead of /home/user/videos/Youtube (notice lowercase videos dir) in auto generated config

My bad linked the wrong thing, edited :D
For me xdg-user-dir VIDEOS returns /home/user/videos


Catch extra exceptions when server connection closes

Catches more exceptions that can happen when the server shuts down.

Description
Currently the only exception that is caught when a connection to the server fails is aiohttp.ClientResponseError. However, we've observed two others occuring:
aiohttp.client_exceptions.ServerDisconnectedError
aiohttp.client_exceptions.ClientOSError
This PR catches them, and handles them the same way as the previous error.
Issues/PRs

Issues:
#204

Related:

Type of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[X ] The code addresses only the mentioned problem, and this problem only
[X ] I think the code is well written
[ ] Unit tests for the changes exist
[ ] Documentation reflects the changes
[ ] If you provide code modification, please add yourself to CONTRIBUTORS.txt

Closed in favor of https://github.com/nolar/kopf/pull/507


externalId is missing in type definitions of getUserList

[x] Review the documentation: https://docs.clerk.dev/
[x] Search for existing issues: https://github.com/clerkinc/javascript/issues
[x] Go through package changelog files.
[x] Provide the Frontend API key from your application dashboard: clerk.learnarena.com

Package + Version

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/clerk-sdk-node
[x] @clerk/backend
[ ] other:

Version:
2.12.1

Browser/OS
Chrome latest
Description
The backend API supports the external_id query parameter on the GET /v1/users endpoint, but it is missing on the type definition of the SDK.

@IGassmann Thanks for reporting this. We have updated the type definitions in the SDK and it should be available in our next release.


Add Default Sharing Token

Hi bro
I add a endpoint to share payment between IBN account that register in Asan Pardakht .
I added it from documented Asan Pardakht share with me and of course test it with real parameter and its worked.

Hey, I made it in another way.
Try it and let me know is it fine for your use case?
Thanks


netcdf: 4.6.0 -> 4.6.1

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/bin/nc-config --help got 0 exit code
ran /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/bin/nc-config --version and found version 4.6.1
found 4.6.1 with grep in /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1
directory tree listing: https://gist.github.com/74ab9782611602d870caba813c09697e

Success on x86_64-darwin (full log)
Attempted: netcdf
Partial log (click to expand)

make[3]: Leaving directory '/private/tmp/nix-build-netcdf-4.6.1.drv-0/netcdf-4.6.1'
make[2]: Leaving directory '/private/tmp/nix-build-netcdf-4.6.1.drv-0/netcdf-4.6.1'
make[1]: Leaving directory '/private/tmp/nix-build-netcdf-4.6.1.drv-0/netcdf-4.6.1'
post-installation fixup
gzipping man pages under /nix/store/v8407ssy7227jhgq2fah6l4hva4mvmfh-netcdf-4.6.1/share/man/
strip is /nix/store/vb6mgjqgd9h11nbv60fvrp1ls9nxck2y-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/v8407ssy7227jhgq2fah6l4hva4mvmfh-netcdf-4.6.1/lib  /nix/store/v8407ssy7227jhgq2fah6l4hva4mvmfh-netcdf-4.6.1/bin
patching script interpreter paths in /nix/store/v8407ssy7227jhgq2fah6l4hva4mvmfh-netcdf-4.6.1
/nix/store/v8407ssy7227jhgq2fah6l4hva4mvmfh-netcdf-4.6.1/bin/nc-config: interpreter directive changed from " /bin/sh" to "/nix/store/8hdxjv3m14jgdnawi5l0mz2h59qkk2g9-bash-4.4-p19/bin/sh"

Success on x86_64-linux (full log)
Attempted: netcdf
Partial log (click to expand)
shrinking /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/bin/ncdump
shrinking /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/lib/libnetcdf.so.13.1.1
shrinking /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/lib/libbzip2.so
shrinking /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/lib/libmisc.so
gzipping man pages under /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/lib  /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/bin
patching script interpreter paths in /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1
/nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1/bin/nc-config: interpreter directive changed from " /bin/sh" to "/nix/store/q1g0rl8zfmz7r371fp5p42p4acmv297d-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/zchkzwsgsnl8aqbxzvi7hh93bdfl4759-netcdf-4.6.1...

Success on aarch64-linux (full log)
Attempted: netcdf
Partial log (click to expand)
shrinking /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1/lib/libbzip2.so
shrinking /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1/lib/libmisc.so
shrinking /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1/lib/libnetcdf.so.13.1.1
gzipping man pages under /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1/share/man/
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1/lib  /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1/bin
patching script interpreter paths in /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1
/nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1/bin/nc-config: interpreter directive changed from " /bin/sh" to "/nix/store/3gg2p6n4kv2f0lsxd41f5iz1ivkbzyzr-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1...
/nix/store/09anigc7d9j4sd5zkq6ckd66my5gddnf-netcdf-4.6.1


Bump identity-apps component versions

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6395936370

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6395936370
Status: success

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6413653134

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6413653134
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6415120298

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6415120298
Status: success

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6490141820

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6490141820
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6497918337

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6497918337
Status: success

Versions are already updated.


Specify foldername in quick_scan_policy and add API key authentication

Hi @kost
Thanks for merging my previous PR. Just wondering if you want to also merge this one and update the gem?
(Otherwise I'm happy to push a new gem and use that)
Thanks
Patrick

replaced with #6


Refresh

Thats a lot of code! :+1:

Resreshing 🍺
On Tue, 15 Mar 2016 at 8:11 PM, Hal Henke<EMAIL_ADDRESS>wrote:

Thats a lot of code! [image: :+1:]
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/in-flux/component-router/pull/86#issuecomment-196731562


Bug: "argument list too long" with large EXCLUDED_SOURCE_FILE_NAMES build setting

Describe the bug
Xcode build fails due to error "/bin/sh argument list too long"
Reproduction steps

On a large iOS project using rules_ios, run rules_xcodeproj's xcodeproj with focused_targets attribute (pass the top level target to focused_targets).
Open Xcode and run build (command + b)

Expected behavior
Build completes successfully
Versions

Bazel: 6.0.0
rules_xcodeproj: 1.2.0
rules_apple: n/a
rules_swift: n/a

Additional context
I noted two things in the generated project.pbxproj file:

Both EXCLUDED_SOURCE_FILE_NAMES and INCLUDED_SOURCE_FILE_NAMES build settings values have too long values resulting to error /bin/sh argument list too long.
Most of the files included in the build settings values EXCLUDED_SOURCE_FILE_NAMES and INCLUDED_SOURCE_FILE_NAMES are .strings resource files.

https://github.com/buildbuddy-io/rules_xcodeproj/commit/71bd29e747a64aa9d9c815e307de8fd29bda0b8f should fix your specific issue while using BwB mode. For projects where BwX mode works, there is still an issue of many resource files causing an issue. I'm trying to think of a way address that.

@yongjincho92 Did the change to use compiler param files fix this for you?

@brentleyjones Yeah it does fix the issue! Thanks


Fix the Edit path and RESTful handling of ems_network

Edit of Network Manager from the Summary page was causing a routing exception since it was not being handled in a RESTful way, which was fixed in this PR.


[Bug]: Dropdown Menu - clicking on trigger when modal false causing menu to close and reappear

Environment
Developement/Production OS: Windows 10 19043.1110
Node version: 16.0.0
Package manager<EMAIL_ADDRESS>Radix Vue version: 1.0.0
Vue version: 3.0.0
Nuxt version: 3.0.0
Nuxt mode: universal
Nuxt target: server
CSS framework<EMAIL_ADDRESS>Client OS: Windows 10 19043.1110
Browser: Chrome 90.0.4430.212

Link to minimal reproduction
https://codesandbox.io/p/devbox/radix-vue-dropdownmenu-jr3jvj?file=%2Fsrc%2FApp.vue%3A42%2C27&layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522clqnjsges0005356b32ub9ocw%2522%252C%2522sizes%2522%253A%255B70%252C30%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522clqnjsger0002356bs5ubnefg%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522clqnjsger0003356b67ydicwe%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522clqnjsger0004356but0k2ir1%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clqnjsger0002356bs5ubnefg%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clqnjsger0001356b75yvdt6q%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252Findex.html%2522%252C%2522state%2522%253A%2522IDLE%2522%257D%252C%257B%2522id%2522%253A%2522clqnju1nh006g356bxqcgw90f%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522initialSelections%2522%253A%255B%257B%2522startLineNumber%2522%253A42%252C%2522startColumn%2522%253A27%252C%2522endLineNumber%2522%253A42%252C%2522endColumn%2522%253A27%257D%255D%252C%2522filepath%2522%253A%2522%252Fsrc%252FApp.vue%2522%252C%2522state%2522%253A%2522IDLE%2522%257D%255D%252C%2522id%2522%253A%2522clqnjsger0002356bs5ubnefg%2522%252C%2522activeTabId%2522%253A%2522clqnju1nh006g356bxqcgw90f%2522%257D%252C%2522clqnjsger0004356but0k2ir1%2522%253A%257B%2522id%2522%253A%2522clqnjsger0004356but0k2ir1%2522%252C%2522activeTabId%2522%253A%2522clqnjt9to003v356bo8mxjlcx%2522%252C%2522tabs%2522%253A%255B%257B%2522type%2522%253A%2522TASK_PORT%2522%252C%2522taskId%2522%253A%2522start%2522%252C%2522port%2522%253A5173%252C%2522id%2522%253A%2522clqnjt9to003v356bo8mxjlcx%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522path%2522%253A%2522%252F%2522%257D%255D%257D%252C%2522clqnjsger0003356b67ydicwe%2522%253A%257B%2522id%2522%253A%2522clqnjsger0003356b67ydicwe%2522%252C%2522activeTabId%2522%253A%2522clqnjt7ms002v356bdyux47km%2522%252C%2522tabs%2522%253A%255B%257B%2522type%2522%253A%2522TASK_LOG%2522%252C%2522taskId%2522%253A%2522start%2522%252C%2522id%2522%253A%2522clqnjt7ms002v356bdyux47km%2522%252C%2522mode%2522%253A%2522permanent%2522%257D%255D%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Atrue%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
Steps to reproduce
Add :modal="false" to DropdownMenuRoot
Describe the bug
When the dropdown menu is open, clicking the trigger immediately closes and reopens the menu. It happens only if  :modal="false" is added to DropdownMenuRoot
Expected behavior
When dropdrown menu is open, clicking trigger shouldn't reopens the menu
Context & Screenshots (if applicable)
No response

Same issue with me

same issue ,also i expect that clicking the trigger when menu is open it should close it

thanks guys! i will prioritize this next week 😁

It seems that after these changes, some functionality was broken, because now I can open more than one dropdown at a time. For some reason, they do not close when interacting outside.

I propbably should open the issue, but I'm too lazy to create a reproduction in the sandbox right now

@MellKam  same issue happened with me

Thanks @MellKam  @peter-emad99 !


Indentation consistency

There's some inconsistency in regards to the indentation; so here's a fix for that.

Hi there, thanks for the heads-up. We have made a commit to fix the indentation (it was easier to double-check this locally). I'm going to close this pull request as we have included it in commit#4. Thanks!


no_mixed_echo_print may ignore one-line structure body

Hi and thanks for the great tool!
Bug report
It seems I've faced with some no_mixed_echo_print fixer (default config) malfunction. I've narrowed it down to this:
Given:
file.php (1):
<?php
	function showSelectOnly($name, $data, $value = null)
	{
		print '<select name="'.$name.'">';
		foreach ($data as $id => $option)
			print '<option value="'.$id.'"'.(
				$value == $id ? ' selected' : ''
				).'>'.$option.'</option>';
		print '</select>';

	}
?>

.php_cs.dist:
<?php

$finder = PhpCsFixer\Finder::create()
    ->in(__DIR__)
;

return PhpCsFixer\Config::create()
    ->setFinder($finder)
    ->setRules([
        'no_mixed_echo_print' => ['use' => 'echo'],
    ])
    ->setLineEnding("\n")
;

Fix attempt:
php-cs-fixer fix  -vvv file.php
PHP CS Fixer 2.18.3 Remote Void by Fabien Potencier and Dariusz Ruminski (ab99202)
Runtime: PHP 7.4.15
Loaded config default from "<masked>/.php_cs.dist".
Using cache file ".php_cs.cache".
Paths from configuration file have been overridden by paths provided as command arguments.
F
Legend: ?-unknown, I-invalid file syntax (file ignored), S-skipped (cached or empty file), .-no changes, F-fixed, E-error
   1) file.php (no_mixed_echo_print)

Fixed all files in 0.008 seconds, 12.000 MB memory used

Result (no_mixed_echo_print alone):
file.php
<?php
	function showSelectOnly($name, $data, $value = null)
	{
		echo '<select name="'.$name.'">';
		foreach ($data as $id => $option)
			print '<option value="'.$id.'"'.(
				$value == $id ? ' selected' : ''
				).'>'.$option.'</option>';
		echo '</select>';

	}
?>

But if I add braces for foreach before fixing, then result is correct: all three print replaced by echo.

At the same time if I add braces fixer (config from Symfony rules set):
.php_cs.dist
<?php

$finder = PhpCsFixer\Finder::create()
    ->in(__DIR__)
;

return PhpCsFixer\Config::create()
    ->setFinder($finder)
    ->setRules([
        'no_mixed_echo_print' => ['use' => 'echo'],
        'braces' => ['allow_single_line_anonymous_class_with_empty_body' => true, 'allow_single_line_closure' => true],
    ])
    ->setLineEnding("\n")
;

then
Result of fixing code (1) (no_mixed_echo_print + braces) is:
file.php
<?php
	function showSelectOnly($name, $data, $value = null)
	{
	    echo '<select name="'.$name.'">';
	    foreach ($data as $id => $option) {
	        print '<option value="'.$id.'"'.(
				$value == $id ? ' selected' : ''
				).'>'.$option.'</option>';
	    }
	    echo '</select>';
	}
?>

So, braces were added, but it is still print inside. Also indentation {Tab}s was partially replaced by {Space}s.

Hi @VBelozyorov, thanks for a report, I've prepared PR ☝🏼 to fix this, would be great if you can take a look and make a review.

Thank you @kubawerlos!
I can confirm that problems with NoMixedEchoPrintFixer described here are eliminated in your PR.