[Bug]: Docker Pull Command Fails When Trying to Update Coolify

Is there an existing issue for this?

[X] I have searched the existing issues

Example public repository
coolify
Description
When I click on the update button from the Coolify dashboard, I get the following error:
Command failed with exit code 1: docker pull coollabsio/coolify:3.12.29
Error response from daemon: manifest for coollabsio/coolify:3.12.29 not found:
manifest unknown: manifest unknown:

I assume this is because the registry for the docker images was changed, and now the command can't find it. All this because Docker decided to delete Open Source organizations.
My question is how do I change the registry so the docker pull command will work, so I can update my instance of Coolify?

Steps To Reproduce

Login to Coolify dashboard.
Click on the pink update button.

Version
3.12.27

Ah right! This could be solved by manually running the installation command on your server, that will update your Coolify.
But I copied the latest version from ghcr to dockerhub, so the button should work!


chore: fixed incorrect use of create namespace

Partially closes #3465 (documentation update needed)

As explained in https://github.com/shellhub-io/shellhub/issues/3465#issuecomment-1906284556, this pull request potentially introduces a significant number of backwards compatibility issues. For that reason, we are closing this. However, thank you for your contribution!


metrics server starts even without --metrics.enabled flag

As reported by a discord member in the lodestar channel.

Describe the bug

Expected behavior

Steps to Reproduce

Screenshots

Desktop (please complete the following information):

OS:
Version:
Branch:
Commit hash:

I've seen that too, we should review and ensure it's not run nor we log that message then

@dapplion  this is coming from: https://github.com/ChainSafe/lodestar/blob/4639d98fe99bdbb5b01998e8d106f79f533f09fc/packages/cli/src/networks/index.ts#L60-L77
when beacon node options are created.
https://github.com/ChainSafe/lodestar/blob/4639d98fe99bdbb5b01998e8d106f79f533f09fc/packages/cli/src/config/beaconNodeOptions.ts#L27
I think we should only build eth1 options data in the getNetworkBeaconNodeOptions and remove the others: metrics,api, network as if any default is required they should come from defaultOptions.

getNetworkBeaconNodeOptions

Yeah it's odd, we should remove from that function lines
https://github.com/ChainSafe/lodestar/blob/4639d98fe99bdbb5b01998e8d106f79f533f09fc/packages/cli/src/networks/index.ts#L64
https://github.com/ChainSafe/lodestar/blob/4639d98fe99bdbb5b01998e8d106f79f533f09fc/packages/cli/src/networks/index.ts#L69
And add the flag to the default docker-compose files


Not working properly in work networks (diffrent users with same ip)

this module doesn't work properly in work networks and throttles all requests from different clients with same IP.
I mean this module doesn't care about sessions and looks at all users in workgroups with the same eye.
users don't need to login in my project so I didn't used Identity to have any client Key.
Is there any option to solve this problem ??
thanks
alireza

If can't differentiate clients then there is no way to apply a rate limit on them.


Change default version from 5.1.34 to 5.1.38.

Proposed fix for Module defaults to http://cdn.mysql.com/Downloads/Connector-J/mysql-connector-java-5.1.34.tar.gz , which is a 404
Change default version from 5.1.34 to 5.1.38. 5.1.38 is listed as the default version https://dev.mysql.com/downloads/connector/j/

thanks @stefanlasiewski!


Separate Twitch chat from clan chat

Twitch chat, by default, currently uses the same colour as clan chat on fixed or a difficult to read red on transparent. When you change the colour for transparent clan chat, both change.
I'd like it if Twitch chat had it's own colour options to keep it separate and distinguishable from clan chat, while also allowing us to make it readable on transparent chat.

Updated the title

Deathbeam is this issue still open?


Why not use CNContact?

It seems like the updated way to do this would be to use CNContact... am I wrong?

Hey @inPhilly, you are not wrong, this library was built before that and never updated.


C++ Windows - Too Many Heap Sections

Checking process manager:

It seems that it reaches 1gb in a 32-bit process and probably that's why it is exiting with a fatal. So it seems that GC is not working or not working properly.

Try compiling with ‑DLARGE_CONFIG

Or it's not working properly, could also be the case. As I said: Will look into it later :)

Sure. Don't worry. I'm just writting here my findings.
In this case, the benchmark creates tons of objects, without actually storing them. So the pointer is replaced in each iteration, and should be disposed. If not immediately, at least before exhausting memory with the GC.
benchmark("Create Instances2", new Task() {
	@Override
	public int run() {
		int out = 0;
		String s = "test";
		for (int n = 0; n < 100000; n++) {
			MyClass myClass = new MyClass(s);
			out += myClass.b;
		}
		return out;
	}
});

Fixed: https://github.com/jtransc/jtransc/commit/5cc2a9af1c937b2ff5796948177140a3d2eff583
Also now runs faster :)

Great that you fixed i!


update role creation for api-gateway tutorial

Issue #, if available:
Description of changes:

Update wording of docs to match the current wording in IAM
Clarified that user needs to add the custom policy creation on the JSON tab
Added missing steps

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for providing the tutorial steps to create the execution role.


Getting an extraneous CR on CREATE TABLE line on OSX if original dump is CRLF.

awk version 20070501
Mac OS X 10.15.1

I'm seeing CRLF on CREATE TABLE opening lines, and only those lines, with windows-created (CRLF) files converted on OSX. The CRLF is replaced with (correctly per awk's RS) LF on subsequent lines within each CREATE statement, but reappears on each subsequent CREATE.
Likely not a big deal, but for the sake of detail, I thought I'd point it out.
Example:
CREATE TABLE IF NOT EXISTS `myveryowntable` (<CRLF>
  `ID` INT(11) NOT NULL AUTO_INCREMENT,<CRLF>
  `User_ID` INT(11) NOT NULL,<CRLF>
  PRIMARY KEY (`ID`)<CRLF>
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4;<CRLF>

CREATE TABLE IF NOT EXISTS `myveryowntable` (<CRLF>
  `ID` integer NOT NULL PRIMARY KEY AUTOINCREMENT<LF>
,  `User_ID` integer NOT NULL<LF>
);<LF>

PS Also note that commas are moved to follow the new line. This is independent of the newline character in the original dump. Not a real problem, just an aesthetic note.

Yeah, it's known, that CRLF is not treated in a special way - sqlite is very generous in this regard, so I don't think it's something we should change :wink:.

PS Also note that commas are moved to follow the new line. This is independent of the newline character in the original dump. Not a real problem, just an aesthetic note.

Yes, this is on purpose (it simplifies the script considerably). The output shall be a valid sqlite input and not something "pleasant to see" :wink:.
Feel free to close this issue if you don't have any actual practical issues with that.


Switching to en-GB dictionary still accepts American English

After adding setting:
"cSpell.language": "en-GB"
the spelling color is still accepted. I tried commenting out the setting and starting again and it then accepted colour.
However on another PC it correctly marked color as a mistake, but then suggested color as acceptable in the list (color, colon, colour, etc)

I am guessing you might need up update the Spell Checker extension.  Can you check the version?

There's something strange going on. On my laptop the extension is on 0.17.2, but on my desktop is stuck on 0.14.8. After uninstalling and reinstalling the extension it is now 0.17.2.  However, this hasn't fixed the issue. I also noticed that I had "color" in the "cSpell.flagWords" list. After removing it was accepted as valid. So there appears to be two issues:

Either it isn't switching dictionaries or the GB dictionary has "color" in it.
Words in the flagWords list are being offered as spelling corrections. I have confirmed this by adding "these" to the list. "these" was marked as a spelling mistake, and top of the list of suggestions was these.

There are two different issues here.

Color is appearing even though it is not a British English word.
Even though color is not in the British English dictionary, it is in other dictionaries like CSS and HTML.  Depending upon the type of file you are editing, different dictionaries are loaded.  This can cause American english words to be included because they are valid keywords.

Words in the forbidden list are being suggested (because the are in one of the active dictionaries).
This is a valid issue.  It is a bit strange from a usability perspective.  The issue is that forbidden words are not filtered out of the suggestion list before it is presented.  If you think this is a big issue, please create a new issue.  It is a perfect candidate for a pull request.

Same issue, 1.3.9 extension running on VSCode 1.16 (x64).

What language local does it show when you right click in your .tex document and choose Show Spell Checker Configuration Info?

or

or

Visual Studio Code 1.18.1 x64
Code Spell Checker 1.6.0
not able to activate en-gb after any manipulations with setup, dictionary always disabled.
What should I do?

There is a bug in VS Code that caches old settings. You have to restart VS Code. ☹
Once the setting has been set, it will be possible to change it with the UI.

So what should I do?
I updated "cSpell.language": "en,en-GB", I have exit VS Code. Start again, open MarkDown file and I see that I don't have active en-Gb dictionary

It is acting like another cspell.json file is blocking the setting.
Try turning on / off Folder settings and toggle the en_GB value.
This should clear local cspell.json files. If that does not work, look for a cspell.json file in the workspace folders. It can be at the root of the folder or inside .vscode.

I have played with switch off/on — no reaction.
I have found two cspell.json

Content is
// cSpell Settings
{
// Version of the setting file.  Always 0.1
"version": "0.1",
// language - current active spelling language
"language": "en",
// words - list of words to be always considered correct
"words": [
"mkdirp",
"tsmerge",
"githubusercontent",
"streetsidesoftware",
"vsmarketplacebadge",
"visualstudio",
"languageclient",
"langs"
],
// flagWords - list of words to be always considered incorrect
// This is useful for offensive words and common spelling errors.
// For example "hte" should be "the"
"flagWords": [
"hte"
]
}
and
// cSpell Settings
{
// Version of the setting file.  Always 0.1
"version": "0.1",
// language - current active spelling language
"language": "en",
// words - list of words to be always considered correct
"words": [
"Pausable",
"Observables",
"Streamable"
],
// flagWords - list of words to be always considered incorrect
// This is useful for offensive words and common spelling errors.
// For example "hte" should be "the"
"flagWords": [
"hte"
]
}
which of them should I edit and how?

@svasil Those files do not need to be changed.
Can you make sure you do not have any errors in your settings.json files?
The best way is to open VS Code settings and look for any errors.

I have message "No problems have been detected"

I am a bit stumped on why it is not working for you. When I get the chance, I'll try adding some more logs to help with hunting down this sort of issue.

Problem was solved in VS Code 1.19.0

@svasil Great to hear!

@BillDenton I'm going to close this issue for now.


Scope Enhancement - Fixed lines in Device Location APIs

What type of PR is this?

enhancement/feature

What this PR does / why we need it:
Include support for fixed lines in Device Location APis
Which issue(s) this PR fixes:

Fixes #91

@jgarciahospital before merging, just wanting to confirm this still needs to go through the TSC?
Stage C: API proposal & voting decision

@jgarciahospital before merging, just wanting to confirm this still needs to go through the TSC?
Stage C: API proposal & voting decision

Enhancements need to be accepted by the proper API subgroup (proposal opened in https://github.com/camaraproject/DeviceLocation/issues/271)
When accepted, we raise it to TSC and merge can be done.


New version: KuaiFan.DooTask version 0.39.97

Created by 🥟 Dumplings in workflow run #9473.

Log
Updated: 0.39.88 → 0.39.97
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -2145844844
2024-11-11 08:26:24.861 [CORE] DeliveryOptimization error: 0x80190194, extended error: 0x00000000
2024-11-11 08:26:24.899 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!00007FFBA3CCA465: (caller: 00007FFBA3CCAD01) Exception(1) tid(414) 80190194 Not found (404).
2024-11-11 08:26:25.453 [CORE] Download request failed. Returned status: 404
2024-11-11 08:26:25.453 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(148)\WindowsPackageManager.dll!00007FFBA3CC43D6: (caller: 00007FFBA3CC5C49) Exception(2) tid(414) 80190194 Not found (404).
2024-11-11 08:26:25.622 [CLI ] Failed to download, waiting a bit and retry. Url: https://github.com/kuaifan/dootask/releases/download/v0.39.97/DooTask-v0.39.97-win-x64.exe
2024-11-11 08:26:27.924 [CORE] DeliveryOptimization error: 0x80190194, extended error: 0x00000000
2024-11-11 08:26:27.924 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!00007FFBA3CCA465: (caller: 00007FFBA3CCAD01) Exception(3) tid(414) 80190194 Not found (404).
2024-11-11 08:26:28.021 [CORE] Download request failed. Returned status: 404
2024-11-11 08:26:28.021 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(148)\WindowsPackageManager.dll!00007FFBA3CC43D6: (caller: 00007FFBA3CC5C49) Exception(4) tid(414) 80190194 Not found (404).
2024-11-11 08:26:21.095 [CORE] DeliveryOptimization error: 0x80190194, extended error: 0x00000000
2024-11-11 08:26:21.130 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!00007FFBB3DFA465: (caller: 00007FFBB3DFAD01) Exception(1) tid(1998) 80190194 Not found (404).
2024-11-11 08:26:21.763 [CORE] Download request failed. Returned status: 404
2024-11-11 08:26:21.763 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(148)\WindowsPackageManager.dll!00007FFBB3DF43D6: (caller: 00007FFBB3DF5C49) Exception(2) tid(1998) 80190194 Not found (404).
2024-11-11 08:26:22.014 [CLI ] Failed to download, waiting a bit and retry. Url: https://github.com/kuaifan/dootask/releases/download/v0.39.97/DooTask-v0.39.97-win-x64.exe
2024-11-11 08:26:24.330 [CORE] DeliveryOptimization error: 0x80190194, extended error: 0x00000000
2024-11-11 08:26:24.330 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!00007FFBB3DFA465: (caller: 00007FFBB3DFAD01) Exception(3) tid(1998) 80190194 Not found (404).
2024-11-11 08:26:24.397 [CORE] Download request failed. Returned status: 404
2024-11-11 08:26:24.397 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(148)\WindowsPackageManager.dll!00007FFBB3DF43D6: (caller: 00007FFBB3DF5C49) Exception(4) tid(1998) 80190194 Not found (404).

Automated error analysis suggests -2145844844 may mean Not Found (404).
Automated error analysis suggests 80190194 may mean Not found (404).
(Automated response - build 897.)

@wingetbot run


How do I setup an environment for openhands resolver?

What problem or use case are you trying to solve?
I'm trying to use openhands with R.  Claude can program in R just fine, but it burns a boatload of tokens trying to install the R packages needed for my repo.  I already know how to do this, so I want to setup an R environment manually and install all the needed packages before running openhands.
However, the openhands resolver seems to be a reusable workflow, so I can't add steps before it to setup the environment it runs in.
Describe the UX of the solution you'd like
Some way to run some steps before the openhands resolver, in order to configure the environment for maximum success.
Do you have thoughts on the technical implementation?
no, github actions confuses the hell out of me
Describe alternatives you've considered
yelling at my laptop
Additional context
I tried adding some steps but got reusable workflows should be referenced at the top-level `jobs.*.uses' key, not within steps

Hmmm @malhotra5 is there a way to achieve this with the current functionality?

@zachmayer makes a good point! We can't support the package installations directly from the Github workflow.
We have the following solution for this, but requires some work from the user:

Setup a custom sandbox, which will contain the appropriate packages.
Pass sandbox_base_container_image="custom-image" parameter through the Github workflow, which will load in the custom sandbox.

Fixed by #5348.


Sign in WebView's lost its state after rotation

Open the WebView for sign in. Then randomly click several links and rotate your device. You will see the web page refreshes back to the sign in page.
Before rotation

After rotation

Yeah, it is the problem. I do not consider the rotation scenario when coding.  It maybe fixed in next version.


chore: remove bindingHolder for proper scoping

I've been perusing lgtm and this one might be in the "makes sense" + 'is minimal" category.
let's see if CI/CD agrees

Coverage remained the same at 92.239% when pulling f48b961ccf2d251988227fec848c5b955654f198 on hairmare:chore/rm-superfluous-trailing-arg into d371c0424a92bda9fde35839a020fe9eec416a60 on knex:master.

thanks!


fix(compiler): initialize compiler with .ts/.tsx/.d.ts files only

Summary

Limit the initial list of file names for LanguageService to only include .ts/.tsx/.d.ts files
Closes #2445
Test plan

Added unit test
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build 660600204

0 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 93.824%

Totals

Change from base Build 660581467:
-0.02%

Covered Lines:
964

Relevant Lines:
994

💛  - Coveralls

Pull Request Test Coverage Report for Build 660600204

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 93.824%

Totals

Change from base Build 660581467:
-0.02%

Covered Lines:
964

Relevant Lines:
994

💛  - Coveralls


keep scroll position of the selected row

Expected behavior and actual behavior.
Actual behavior: I clicked a row(index 6), the selected row became high light with border-color style.After a little while, my app received many grid data,but the selected row scrolling below ,also the high light row index is still 6.
In my app,my grid data maybe thousands of row in one page,so i wish keep the selected row and its high light style when received new data, don't affect checking the selected row data..
Can u tell what can i do ,thx very much
Steps to reproduce the problem.
Specifications like the version of the project, operating system, or hardware.
"canvas-datagrid": "^0.22.12"
"electron": "10.16.0"
"react": "16.8.0"

You can capture the selected rows with this property:
https://tonygermaneri.github.io/canvas-datagrid/docs/#canvasDatagrid.selectedRows
You can set the selected rows with this method:
https://tonygermaneri.github.io/canvas-datagrid/docs/#canvasDatagrid.selectRow
You can detect when changes occur by listening to this event:
https://tonygermaneri.github.io/canvas-datagrid/docs/#canvasDatagrid#event:selectionchanged
If you listen for selection changes, record elsewhere what the selected rows are, then when you change your data, use selectRow to reselect the desired rows, if they still exist.
I hope that answers your question.
Cheers,
Tony


Pre 5 support

Thanks! does the first commit fix the support for 4.2.10 indicated in https://osmocom.org/issues/6203? if yes, please indicate that explicitly in the commit log and add the link to the issue.
Similarly, if the second commit is already sufficient to make the plugin work on 5.1.x, then please state that explcitly in the commit log, stating the exact redmine version that you tested with, and how you tested it. Thanks!

When trying this with redmine 5.0.9 or 5.1.3 I'm getting
redmine-test_1 | NameError: uninitialized constant OpenIdAuthentication
redmine-test_1 |
redmine-test_1 | new_middleware_stack.delete OpenIdAuthentication # Modify the copy
redmine-test_1 | ^^^^^^^^^^^^^^^^^^^^
redmine-test_1 | /usr/src/redmine/plugins/redmine_openid_provider/init.rb:13:in `block in <top (required)>'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:234:in `instance_eval'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:234:in `configure'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/railtie.rb:191:in `configure'
redmine-test_1 | /usr/src/redmine/plugins/redmine_openid_provider/init.rb:11:in `<top (required)>'
redmine-test_1 | /usr/src/redmine/lib/redmine/plugin_loader.rb:31:in `load'
redmine-test_1 | /usr/src/redmine/lib/redmine/plugin_loader.rb:31:in `run_initializer'
redmine-test_1 | /usr/src/redmine/lib/redmine/plugin_loader.rb:108:in `each'
redmine-test_1 | /usr/src/redmine/lib/redmine/plugin_loader.rb:108:in `block in load'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:427:in `instance_exec'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:427:in `block in make_lambda'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:198:in `block (2 levels) in halting'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:604:in `block (2 levels) in default_terminator'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:603:in `catch'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:603:in `block in default_terminator'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:199:in `block in halting'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:512:in `block in invoke_before'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:512:in `each'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:512:in `invoke_before'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/callbacks.rb:105:in `run_callbacks'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/reloader.rb:88:in `prepare!'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/application/finisher.rb:124:in `block in <module:Finisher>'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in `instance_exec'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:32:in `run'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:61:in `block in run_initializers'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/initializable.rb:60:in `run_initializers'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:391:in `initialize!'
redmine-test_1 | /usr/src/redmine/config/environment.rb:16:in `<top (required)>'
redmine-test_1 | /usr/local/bundle/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
redmine-test_1 | /usr/local/bundle/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
redmine-test_1 | /usr/local/bundle/gems/zeitwerk-2.6.18/lib/zeitwerk/kernel.rb:34:in `require'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:332:in `block in require'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:299:in `load_dependency'
redmine-test_1 | /usr/local/bundle/gems/activesupport-<IP_ADDRESS>/lib/active_support/dependencies.rb:332:in `require'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:367:in `require_environment!'
redmine-test_1 | /usr/local/bundle/gems/railties-<IP_ADDRESS>/lib/rails/application.rb:533:in `block in run_tasks_blocks'
redmine-test_1 | /usr/local/bundle/gems/rake-13.2.1/exe/rake:27:in `<top (required)>'
redmine-test_1 | Tasks: TOP => db:migrate => db:load_config => environment
redmine-test_1 | (See full trace by running task with --trace)

The error is quite clear: OpenIdAuthentication is undefined, and in fact it is so since https://github.com/redmine/redmine/commit/9955844da1d209778f06ad9a3583949048b8ee88 was merged ahead of 5.0.0, removing all of the redmine built-in openId client support.
I do not see how your PR could ever have been tested on any version >= 5.0.0 due to this :(

A working, tested fix is now in https://github.com/osmocom/redmine_openid_provider/commit/ba484700a6aa9f061f50c49f4467dd0c7f3db374


Dont wheel local directories

People expect pip install . to work without bumping the version.

:boat: it

Tests are flaking because we assert the ref name:
assert set(frozen_requirements) == set([
    '-e<EMAIL_ADDRESS> # noqa

we get errors like
E         -  'git://github.com/Yelp/dumb-init.git@a9eadb580c0d234fc4090c1bf3f19f8d87bff76b#egg=dumb_init-origin/HEAD',
E         ?                                                                                ^^^^^^^^^^^

or
E         -  'git://github.com/Yelp/dumb-init.git@a9eadb580c0d234fc4090c1bf3f19f8d87bff76b#egg=dumb_init-origin/master',
E         ?                                                                                ^^^^^^^^^^^^^

If there are multiple matching refs, new pip always picks the first alphabetically, but the old pip just arbitrarily returns one: https://github.com/pypa/pip/commit/8d1765476e7da7794390ece9ed5ffe978a2efe96
I think we can just pick a random in-the-middle commit which has not a tag or branch name. Feels kinda yucky though, ideally we'd probably make the git repo ourselves as part of the tests.


fixed android logout

As mentioned in https://github.com/moberwasserlechner/capacitor-oauth2/issues/97 there are currently some issues with the logout functionality. We discovered them only on Android, so this fix is only implementing a solution for Android. Our project uses SalesForce OAuth2 provider.
We found that our users had problems with logout whenever they did not use their system's default browser. For example, if they had configured the Brave browser or Firefox as their default browser. In these cases we were not able to delete the session cookies. This is because this plugin always uses the system browser (usually Chrome), no matter what the user has set as default. But when we open a new browser window using Capacitor, the default browser configured by the user is used. And just not the system browser.
That's why we implemented the logout functionality for Android and our users can now logout correctly on Android. No matter which browser they use and no matter which browser is default on their system.
I implemented the customization in such a way that no changes are necessary for existing applications. The code is fully backward compatible.
If you want to use the new logout functionality, you now have to pass the current accessToken as a second, optional, parameter to the logout(...) function (required by the Android plugin). If you do not pass the parameter, the plugin will behave as before.
I have adapted the documentation as good as possible. If it is incomplete or you want it to be different, please let me know.

Thanks for this PR

You're welcome and I need to thank you for providing such an awesome plugin. :)

I might be wrong but id_token parameter currently has no effect. I have printed the result of call.getData() and it's not there.
For tests purpose I forced id_token with this snippet:
// @ts-ignore
oAuth2AuthenticateOptions.id_token = _accessToken;

That will result into NullPointerException because the logoutUrl is not set in this method:
OAuth2Options buildAuthenticateOptions(JSObject callData)
Also setLogoutUrl(String logoutUrl) is not defined in OAuth2Options.java
Making these changes will end in this error that I could not solve:
java.lang.NullPointerException: Attempt to invoke virtual method 'android.net.Uri$Builder android.net.Uri.buildUpon()' on a null object reference

Just realized the same thing as @Ssnipo. This can't possibly work. It seems to have been merged blindly without any testing.
Considering this has been reported over a year ago, is this lib even maintained anymore?

@zolakt you are very welcome to help maintaining this lib. That's how OSS should work, considering the work for this lib was done in ones spare time.

I feel really sorry about that my implementations didn't work as expected. I used it in a client project back then and it worked there. Probably I had more changes directly in the project that I forgot to check in, I doubt it, but it could be the case. It's too long ago to be sure about.

If I ever get the time, I'll try to help out.

That's exactly the point why no one has fixed it so, I guess. Because we all feel so.

@svzi I think your change back then was fine as well. You tested it and if there is another aspect to it, it simply is a bug or an improvement. So thanks for the PR back then.
@zolakt If you know whats wrong with the docs please at least make a PR for the docs. You have thought into the topic and I myself and others have not.

Honestly I doubt that it ever worked, since there are multiple issues with it. It's more likely that not everything was pushed, back then. The id_token argument that was added isn't parsed properly. Even if it was, it's never used, since it's not included in  OAuth2Options. The setter is missing there, and there is no code that would set it up. It's mentioned in this post https://github.com/capacitor-community/generic-oauth2/pull/233#issuecomment-1542479281. I've came to the same conclusion, and gave up on the same issue.
@moberwasserlechner Here you go https://github.com/capacitor-community/generic-oauth2/pull/278
I removed all mentions of logout() from the examples, and added a section which points to the issue where workarounds are discussed.


adding a work around for new gunicorn

Coverage decreased (-0.08%) to 67.956% when pulling f1d8793c1c15ee1cbd44cb838cac5befcb3754ab on kevgliss:cli into 9d03e75d9b27076b98102b776d4b17be5808cfac on Netflix:master.


initial sketch for session.regenerate() support

This changes how sessions are provided by the middleware in order to
support the ability to use session methods, like regenerate. Crucially
sessions are no longer cloned and instead are stored via a handle. This
handle is defined as Arc<RwLock<async_session::Session>> which allows
us to clone the Arc without cloning the contained session directly.
In order to support better ergonomics we also introduce two new
extractors: ReadableSession and WritableSession. These respectively
enable reading and writing of sessions. Internally they acquire the lock
and expose the session for manipulation.
One downside to this approach is that library consumers now need to use
two separate types as opposed to using async_session::Session via
Extension directly. However the API here now no longer requires
Extension and enforces explicit usage of reading and writing via the
public type interface.
Fixes: #5.

@Ptrskay3 here's what I've put together pursuing the interior mutability option. I'd love your feedback.

I'll go ahead and merge this. Thank you for your great feedback!


How to show warnings?

Description
If a command doesn't terminate in error then lint-staged doesn't display its stdout but I was wondering if there's a way for commands to not terminate in error but still print out warning messages to the screen that lint-staged will display?

We use listr to render the Terminal output, so this seems related: https://github.com/SamVerschueren/listr-update-renderer/issues/10

Alrighty, I'll follow up on that comment. Thanks!

Now that I look at resolveTaskFn, the task we run using listr returns a success message in the form of ${symbols.success} ${linter} passed!. This will probably silence any warning output generated by the task itself, since that is not captured anywhere.
What do you think, @okonet? In the case that a task returns a warning message, should we display that instead of the success message? The question is also how to determine success vs warning output.

For what it's worth, we're considering using this package at my work, and we definitely would prefer to be notified of warnings when making a commit, thus allowing us to amend the commit, or move along.
I see the merit in not showing warnings, so I'd definitely welcome an option to pass a flag to output the warnings the linter picks up.

I’m not sure what’s the purpose of printing something and allowing committing regardless. If you want developers act on warnings you should consider making warning to errors or use eslint argument that exits with non-zero when there are warnings.

I see where you're coming from, and in a perfect world I would elevate these to errors, thus forcing developers to fix the thing that's getting caught by our linter (ESLint).
However displaying warnings with a non-zero exit code is exactly how ESLint operates, and if I were to run a pre-commit hook just running ESLint itself, I'd get the exact behavior I just advocated for aside from the fact it'd run across all files, not just the staged ones.
Like I mentioned, it's not that I'd want it to be a default option or anything (or even taking the position that it'd be ideal), but just having the option would be appreciated, especially as it relates to the behavior of the tools we're interfacing with here.

something like --verbose, which would disable the default success message and instead return the the all value from execa (which includes both stdin and stdout), might work. What do you think, @okonet?

As have been noted already - the point is that at times you may not want to break a commit that abruptly interrupts a developer's workflow and instead you'd want to show a warning so they can take some actions to remediate it in the future.

https://eslint.org/docs/user-guide/command-line-interface

https://eslint.org/docs/user-guide/command-line-interface
--max-warnings

Unfortunately that wouldn't solve the exact parameters mentioned in the initial issue proposal/question:

to not terminate in error but still print out warning messages
Using --max-warnings would generate a non-0 exit code, and error out a pre-commit.

I appreciate you reaching out, though, with a possible solution. If it's impractical to expect this to be implemented, that's fine - honestly it's kind of what I expected to happen, but I did agree with the desire of the initial question that brought us here

I don’t think this is going to be implemented for the reason I mentioned before: i don’t see any practical use case for printing something that needs attention and yet proceed with the commit. Maybe I’m still missing something here but for now I’d close the issue.

Just to clarify: lint-staged should be used as the only quality control tool. It’s designed for a solely purpose of not allowing commits that are considered erroneous.

I don’t think this is going to be implemented for the reason I mentioned before: i don’t see any practical use case for printing something that needs attention and yet proceed with the commit. Maybe I’m still missing something here but for now I’d close the issue.

I don't disagree with your opinion here, I think it's a good point and should be taken seriously, but I did want to point out that sticking to it at the very least is nullifying the native behavior of the commands that are being run.
If a command has the prerogative to regurgitate warning texts with a 0 exit code, it seems unfortunate to accept that behavior to be nullified when running them through lint-staged
But that's ok, agree to disagree, I certainly understand where you're coming from.

@okonet I agree with @JimmayVV. As a user, if I don't see any warning but a green mark, I'll assume everything is doing great. And those warnings can accumulate and spread out all over project with time passes. For me it's not if we are going to take action on those warnings but we are informed so we have a choice.

🚫💩 lint-staged should not be considered as the only code quality tool in your tool belt.  Please use other eslint integrations to see warnings.

Can you give me an example of how to use it as last resort? I'm following examples from README, eg. "*.js": "eslint --fix".

You could use —max-warnings=0 with eslint to stop the commit.

Thanks for the suggestion, but I don't want to stop commit if there are warnings. We have many lint rules and changing that will require many code changes. What I hope to achieve is run linter on staged files, display warnings if any for awareness, commit on no errors.

lint-staged should NOT be used as the only quality control tool. It’s designed for a solely purpose of not allowing commits that are considered erroneous

The project documentation suggests otherwise. I.e. it contains many code style improvement examples as well as image compression example etc.
I would like to use this to run image compression on my images. While doing that i would like to see get report on KB saved per specific image.
I think having ability to report any kind of output of the task would improve versatility of the tool as it would let developer decide what information should be reported while running it.

So I hear the following: you want to print what the command has printed while running. Although it’s not possible during the task running we could collect all the outputs and print them after the run is complete. This is what we do for errors now and that’s what we could do for stdout as well. If the tool prints to it during the execution it will be printed. Is this how everyone imagine it should work? If yes, I’m okay with that change I think. Does anyone want to give it a try and create a PR?

So I hear the following: you want to print what the command has printed while running. Although it’s not possible during the task running we could collect all the outputs and print them after the run is complete. This is what we do for errors now and that’s what we could do for stdout as well. If the tool prints to it during the execution it will be printed. Is this how everyone imagine it should work? If yes, I’m okay with that change I think. Does anyone want to give it a try and create a PR?

Yes! This is exactly what I was advocating for earlier. In general I'd love to see what the outputs were in their entirety when running this package. I know in my use case I guess you could say it defies logic somewhat to want to learn information from the outputs while still allowing a commit to proceed, but indeed that is exactly what I'm an advocate for.
As for you mentioning it can't be done in real time, and instead it will just be printed at the end of the command, that seems perfectly fine by me, and hopefully others as well :)
Thanks for reconsidering!

Hello, just start implementing lint-staged in in our quality process.
Thanks for this really great tool !
It's true that the abilities to display custom/warning message can be very positive/usefull, and respect used linted tools warnings. Like optimisation, not mandatory (not error) but the developper can improve himself on it for instance. More generally it will allow "code is not perfect, but OK for respecting CI rules, but this could be even better!".
My use case for instance: i check deprecated code, it is allowed by CI/code politics, but it is good to inform the developper that in his commit he use code which will no more be compatible on future versions. Just an example. Use case are infinite : ) (trying rules without enforcing them, etc.)
I use npm scripts, shell script, php scripts... so a generic method based on stdout is the best (Listr is npm and restricted to this area so).
Having stdout processed like stderr (stored in buffer displayed at the end) is perfect.
The only thing to be carefull is to remember the task command and append it before it stdout result. It is just a suggestion, but this point is different from error (where only one task is concerned by the stderr output). As some tools could not report their name during execution, it will make output more comprehensible (and avoid du create a wrapper script just to output "hey i am tool1 : )").
It could be something like that, the command executed for the tast between [ ]:
[my-tool]
stdout of this task

[sh -c "some commands"]
stdout of this task

...

[task command] will only be displayed if stdout is not empty.
By activating this verbose option, developper should be able to hide non wanted output specific task by simply redirecting stdout to null, example:
"*.ext": [ "my-command1 1>/dev/null", "my-command2" ]

So only warning of my-command2 will be reported. I think it will work as it (if redirect to null, lint-staged should see nothing to buffer and output at the end of execution), but just to be sure it handle well those cases.
I hope this comment will help to this new feature.
Best regards.

This would also be very helpful for my team. We want to show warnings when running the linter even if there are no errors, just as a "hey, check these out if you can" sort of thing.

This is actually possible already, I've just totally forgot to link to this issue. Using the --verbose flag will always show task output.

The --verbose flag was released in v10.2.0.


Update index.html

Added some punctuation to make all sections consistent. Mostly full-stops but also added a semi-colon and reduced double exclamation marks to just one.

For example, the easy to use, auto converters and multiple error modes sections, already use full stops, but the others do not.

Thanks !!


Update sbt-scoverage to 1.8.2

Updates org.scoverage:sbt-scoverage from 1.6.1 to 1.8.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

labels: sbt-plugin-update, semver-minor

Codecov Report

Merging #257 (ba7fa25) into master (2d51420) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #257   +/-   ##
=======================================
  Coverage   43.72%   43.72%
=======================================
  Files          25       25
  Lines         263      263
  Branches        6        6
=======================================
  Hits          115      115
  Misses        148      148

Continue to review full report at Codecov.

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

Closed in favour of #277


Build error in trunk

Build error in trunk:
CImportExportUIController.cpp: In member function ‘bool CImportExportUIController::loadFromFile(const QString&, const QString&, CEditorScene&, QString*)’:
CImportExportUIController.cpp:263:50: error: cannot convert ‘CEditorScene’ to ‘CNodeEditorScene&’
263 |         return (COGDFLayout::loadGraph(fileName, scene, lastError));
|                                                  ^~~~~
|                                                  |
|                                                  CEditorScene
In file included from CImportExportUIController.cpp:17:
../../src/commonui/ogdf/COGDFLayout.h:32:70: note:   initializing argument 2 of ‘static bool COGDFLayout::loadGraph(const QString&, CNodeEditorScene&, QString*)’
32 |     static bool loadGraph(const QString &filename, CNodeEditorScene &scene, QString* lastError = nullptr);
|                                                    ~~~~~~~~~~~~~~^
Ubuntu 20.04.1 LTS
gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0
QT 5.12.8
Build with OGDF and boost

This had been already reported as #130.

Fixed for 0.7.dev


Reduce warnings/pop-ups in new versions

First off, I love CMake tools, and in my opinion it wouldn't really be possible to use visual studio code for C++ development without it.
One improvement that I think could be made is reducing the warning/pop-ups associated with the extension. They cover up the file tabs and are really annoying and overused by both code and many extensions. In general they make for a bad user experience.
For example, my team maintains a group of desktop machines which are reimaged occasionally, so every time we use a different machine or it gets updated, code thinks that it is showing us the pop ups for the very first time, but in fact it's the millionth time. I know that some options are deprecated, and I know that there is debugger support now.
Trust that the users who actively care about trying out the new features are reading the release notes (especially now that they are integrated directly into code). If not, shame on them, but don't force pop-ups on all of us to try to catch a few disinterested users.

Thanks for the feedback! I'm thinking about adding some sort of telemetry so that I can actually gauge feature usage rather than throwing pop-ups and seeing what happens. I hadn't really considered your use case, but it makes sense that it would get troublesome. I'll remove the pop-ups next release. Until then, it is useful to note that all visible pop-ups/warnings can be dismissed with an escape key press.
Regarding deprecated options, though: I inserted those warnings as a way to ward off potential tickets or issues saying "Option X is suddenly broken." The deprecated option warnings should not appear unless you are actually making use of the bad options, and the options themselves should be completely non-functional. What is your use case that is causing them to appear?

There was a time when our maintainer thought it would be a good idea to enumerate every single option in the user settings, so probably it was actually being set when it shouldn't have been.
Didn't know about the ESC key press.

I was looking into reducing the number of pop-ups, and I found that the main offender, the "Debugging" popup, hasn't been in the code for a few versions. Could you identify what pop-up messages are causing you the most grief? I'll do my best to get them out of the way in the future.

I just noticed the reload this window to enable cmake-server warning, which I'm not using. Also, reloading the window just causes the message to appear again, so that seems more of a bug. Possibly related to 0.9.6 and 0.9.7 failing to allow anything to build. Had to downgrade to 0.9.4 and turn off auto updates to get back to a usable state.

@elalish It is likely that you're using old CMake version. Please see #173 for workaround. I hope fixed version will be released soon.

Closed.
If this issues still persists in the current 0.11.0-beta4, feel free to reopen this one or create a new one.


Fixed spelling and grammatical errors.

The diff still shows the underline is not of the same length as the text above it.

Seems like these issues have been fixed already.


cryptsetup without key explodifies due to keyscript arg in crypttab

I discovered that if one sets none as the key file in crypttab, cryptsetup initramfs script barfs and drops you to initramfs busybox.
When removing the keyscript,tries args (keyscript=/lib/cryptsetup/scripts/passdev,tries=3) then a normal prompt is restored.
I know that, currently, the script doesn't support setting none for a root key--so by deviating from the way the script currently functions regarding crypttab/keying, I brought this on myself. That said, when you get a chance to improve this featureset, this information will be important if you didn't already know.
Thanks.

I'm not understanding what happened here.  Were these manual edits to crypttab after the script was run?
At present, if you turn encryption on there are three key modes that are supported (using AUTO_DISK_PWD):  the code will generate a keyfile for you (this is the default), you can pass a local file to use as the key file, or you can enter a disk password.

the tl;dr is this:
echo "cryptroot UUID=${main_uuid} /dev/disk/by-uuid/${boot_uuid}:root.key luks,initramfs,keyscript=/lib/cryptsetup/scripts/passdev,tries=3${discard_option}" >> /mnt/etc/crypttab - this is inserted by the script, this does not function unless root.key exists (i.e. a password is used) and must be changed to:
echo "cryptroot UUID=${main_uuid} none luks,initramfs,keyscript=/lib/cryptsetup/scripts/passdev,tries=3${discard_option}" >> /mnt/etc/crypttab - this change (removal of the hardcoded keyfile) also fails, I'm unsure of the exact reason but removal of keyscript,tries (let initramfs cryptsetup fall back to defaults) with:
echo "cryptroot UUID=${main_uuid} none luks,initramfs${discard_option}" >> /mnt/etc/crypttab - functions as expected

I accidentally hit "close with comment" reopening

Ok.  This seems to be redundant with #63.  As they are both related to problems when passpharse is used.  I have fixed #63 so this item should be fixed as well.
Closing.


Mac format fix

Addresses issue #32 with regards to blob data. Also fixes make sure the mongo query resolves before ending the test.

@harsimrat99, did you add the bucket as a repository variable? It seems that the test cannot find the bucket parameter. It also might be related to the wrong version of formidable mentioned here though

Hi Simon, I did add the repository variables and I think they should be accessible when running GitHub actions.


add RepoContent.GetContent to replace Decode

This came up when reviewing #323.  I think the best approach here is to have a single method that callers are instructed to use, regardless of the content encoding... they shouldn't have to think about that.
/cc @mjbrender

LGTM

How come PRs are showing up with 0 commits lately?

uh, that's weird.  It had a commit earlier when Glenn reviewed it.  I did delete the branch on my fork, but that shouldn't have made a difference.  GitHub bug maybe?
/cc @fabianperez

Thanks for these improvements and for keeping me in mind @willnorris. 👍


redirect loading bug

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
There is a bug where the page shows the loading checking if user component screen for a second and then immediately redirects to a blank page.

Oh this isn't a bug. I just didn't deploy the backend yet.

yep issue resolved


Can this package automatically reorder with animation when data source is changed?

In my app, I don't need the user to reorder the contents but they are done when some filters are enabled.
Use case:
Grids show cars.
When user presses red color, I want to show only the red colored cars
And I want this filtering to happen in an animated way.
So there might be several cars that are removed from different places of the cars array.
When the filter is removed, all cars should be added in place with ordering animation.
Can I use this package for this need?

There is no provision for order now, considering as enhancement. Thanks @aytunch


New samples

New samples in a repository https://github.com/JetBrains/kotlin-web-site
Files:
docs/topics/control-flow.md

New samples in a repository https://github.com/JetBrains/kotlin-web-site
Commit: https://github.com/JetBrains/kotlin-web-site/commit/e1e97f14c645ac19c1299983a42bee904d13673e
Files:
docs/topics/tour/kotlin-tour-null-safety.md
docs/topics/tour/kotlin-tour-basic-types.md
docs/topics/tour/kotlin-tour-collections.md
docs/topics/tour/kotlin-tour-hello-world.md
docs/topics/tour/kotlin-tour-functions.md
docs/topics/tour/kotlin-tour-control-flow.md
docs/topics/tour/kotlin-tour-classes.md


sort, store and modify all photos on onedrive so that all collaborators can see all photos

https://github.com/NewGraphEnvironment/fpr/issues/32
For now we store report/map photos on github and push only photos tagged with _k_ and crossing_all to Github to reduce repo bloat.  This makes collaboration difficult. Solution for this repo below.
We will now do no photo work (renaming, resizeing, rotating, etc) in the repo itself. All work will be done in onedrive with sync done after each modification
mac - uses rsync
https://www.cherryservers.com/blog/how-to-use-rsync-on-linux-to-synchronize-local-and-remote-directories#:~:text=If the specified destination folder,the directory will be copied.
rsync –a SourceDirectory DestinationDirectory
rsync -av --delete ~/Library/CloudStorage/OneDrive-Personal/Projects/repo/fish_passage_peace_2023_reporting/data/photos/AI/test ~/Projects/repo/fish_passage_peace_2023_reporting/data/photos/
Flags
-a, --archive               archive mode implies recursive and preserves important file attributes such as file permissions, group and user ownership, modification times, symbolic links, etc.
 -v, --verbose               increase verbosity Just to show us what is happening
--delete                delete extraneous files from destination dirs this could be considered risky because repo only photo work will be lost on syncs but allows us to operate so that onedrive is the main and the repo is the replica as we will now do no photo work in the repo.
windows
https://www.ubackup.com/synchronization/windows-sync-two-folders-command-line-5740.html

here are some screenshots to show how this was tested on the mac.
before sync with repo files altered (not something we will do of course)

after sync with fresh change to onedrive file
rsync -av --delete ~/Library/CloudStorage/OneDrive-Personal/Projects/repo/fish_passage_peace_2023_reporting/data/photos/AI/test ~/Projects/repo/fish_passage_peace_2023_reporting/data/photos/

Not surprisingly onedrive seems to be glitching due to massive un-synced changes.  The size of the diffs resulting from thousands of photo renameing and delete operations during repeated testing and delivery of fpr::fpr_photo_rename over the last week has exposed weaknesses in our ability to use this.  Better sooner than later.
We will need to move to dropbox, aws or digital ocean. Dropbox is expenses but has intuitive user interfaces.  We already use digital ocean for overall company file backups and remote servers but amazon is likely the cheapest and perhaps the most flexible solution. Being able to serve the photos directly to reports served on github would be a huge benefit. Storing any photos at all on Github is far from ideal.  lfs for git could be an option but I have had major issues with that in the past so would like to avoid that option.
For now - renamed photos have been moved to new directories in onedrive/2023_data/{region}/photos.  Lets try working from there for now to attempt to get the Phase 1 data submitted for both Peace and Skeena
local onedrive

cloud onedrive

photos sorted using names from dff with fpr_photo_rename and duplicates with fpr_photo_remove_dupes are in onedrive/projects/2023_data/peace/photos
photos moved from {onedrive}/projects/2023_data/peace/photos to {onedrive}/Projects/PSCIS/phase1_submissions/2023/peace with fpr_photo_paths_to_copy ready for PSCIS submission - Phase 1.
Note that new photos can be added to the directories for submission to PSCIS by usingfpr_photo_paths_to_copy to move from {onedrive}/projects/2023_data/peace/photos to {onedrive}/Projects/PSCIS/phase1_submissions/2023/peace once you have added the 'k' tag, a custom tag via the 'tag_add' param or by tagging with any of the identified tags in fpr::fpr_xref_photo_order. See ?fpr::fpr_xref_photo_order for info and run fpr::fpr_xref_photo_order for a visual

rsync -av --delete ~/Library/CloudStorage/OneDrive-Personal/Projects/2023_data/peace/photos  ~/Projects/repo/fish_passage_peace_2023_reporting/data/

need to build crossing_all.JPG photos on onedrive so that we don't erase them in the local repo when we run a sync

Some things to note when using rsync on mac:

you must have the OneDrive folder on your local device with the source/destination folder in it. To add a shared folder to "my files" look here
When calling rsync, you must be outside of the source folder. An example from blackwater_2022_hep#1 on my machine looks this this, I am in my home directory:
~ rsync -av OneDrive/blackwater_2022_hep/data Projects/repo/blackwater_2022_hep/data

When calling rsync, you must be outside of the source folder

or do we need to just call full (absolute or complete vs relative) path names (right-click + option + copy {directory location} as pathname) (ex. /Users/airvine/Library/CloudStorage/OneDrive-Personal/Projects/2023_data/skeena/fish - ie. home directory location is included in the path name)

closing since this is happening


Add support for including Password Rotation in the Password Policy Report

Code of Conduct

[X] I have read and agree to the project's Code of Conduct.
[X] Vote on this issue by adding a 👍 reaction to the original issue initial description to help the maintainers prioritize.
[X] Do not leave "+1" or other comments that do not add relevant information or questions.
[X] If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Description
Add support for including a Password Rotation in the Password Policy report alongside expiration, complexity, and account lockout.
This information would come from SDDC Manager to reflect the status of password rotation:

account and type
enabled / disabled / not applicable
rotation interval
last rotation
next rotation
comparison with password expiration to expose conflict
etc

Use Case(s)
Provides a more complete view for the status of password rotation alongside expiration, complexity, and account lockout.
Potential Configuration
To be determined.
References
No response

Will rely on PowerVCF v2.3.0 release with the inclusion of https://github.com/vmware/powershell-module-for-vmware-cloud-foundation/pull/191.

Note
This may move to the proposed VMware.CloudFoundation.PasswordManagement module and the deprecation and/or removal of the password policy report within this module.

Marking as closed. The password policy information is moving to the VMware.CloudFOundation.PasswordManagement module.
Will add this context to an issue for that module.


floating point math bug

experience gained from fishing is 10x experience earned
0.001 XP gained -> 0.01 XP earned

I think i know whats causing this. I'll aim to fix this in v1.0.2

ok, turns out when displaying caught xp it divided the rarity by 100 (correct value) BUT when updating the users xp, it divided by only 10. When updating XP it divides by 100 now.
Thanks for catching this, i would've never noticed this otherwise
Are you on the support server? You've earned the contributor role


Theme sync with system #216

Sync with system and automatically switch between light and dark themes.

Nice commit! 👌 (Whoops I merged it wrong)


gtk-single-instance=true and opening new window doesn't use up to date config

When setting gtk-single-instance=true and opening a new window via the DE, it will use a non-up to date config
reproduction:

set gtk-single-instance=true on some config X and reload config
change some config value to config Y
open new ghostty via DE
new ghostty will use config X instead of Y

Opening a new window via the original already open window will use config Y
Closing all ghostty windows(but not the background process left behind) and opening a new one, will use config X.
Expected behavior: all tabs and windows opened after the config has changed(especially after it has been reloaded), should use config Y

this issue somehow just disappeared after trying it again, after multiple times that it did happen, unsure what happened


Fix bug: Clashing lesson time prevents user from editing lesson time for the same student

Problem
If editing the lesson time for the same student from one to another with the new lesson time overlapping with the previous, the edit is not permitted.

Fixed with #197


Binding anonymous with name for functionExpression and classExpression

Aside from the change in unit-test; the RWC has no regression

:+1:


Change to using sobek instead of goja

We are moving to a fork of goja under grafana org called sobek.
More info in:

https://github.com/grafana/k6/issues/3772
https://github.com/grafana/k6/issues/3773

I am actually closing this PR.
The PR was created with a script for each extension in the k6 docs in order to update to sobek, before that becomes a breaking change.
AS this extension is an output it does not use sobek, so we just update k6.
This isn't really needed for updates, so I will just close this.
If the maintainers want - they can update k6 - maybe even to the just released v0.52.0


Update build

Try to fix linux build that failed last time.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Whenever possible python packages should use pip. See https://conda-forge.org/docs/meta.html#use-pip


"on-click" not working

config
"network": {
    "interface": "wlp1s0",
    "format-wifi": "{essid} ({signalStrength}%)",
    "format-disconnected": Disconnected",
    "tooltip": false,
    "on-click": "$HOME/scripts/wifi"
},

EDIT
It seems that "on-click" isn't working on any of the modules for some reason. I had several set for custom modules and they suddenly aren't working.
All scripts invoke sudo as they aren't compatible with pam. They worked as recently as a few days ago. There are no journalctl errors listed. I'm not sure how to debug this.
I've noticed that everytime I adjust style.css and then refresh sway, the entire system freezes - again with no journalctl errors listed. I have to hard reset each time this happens. If I log out of sway and log back in this doesn't happen. I can otherwise refresh sway repeatedly and it won't crash unless I've edited style.css.

After restarting I used a different script and it's working properly. There must be something conflicting in my sway config that isn't allowing it to pop-up correctly. This is not a bug in waybar - sorry.


Support timeout for request

Requirements
We should support timeout when calling the API to be sure to not block the process for ever.
We should have an option to set the timeout, something like --timeout and -t.

hi @thomaspoignant,
i am interested to work on this issue. let me know if the issue is still up for grabs.

Hi @andrysds yes this is still available, go head.
Thanks a lot for your interest it is awesome to have more people on board


The list in antidote/lessons/lesson-14/stage1/list.yaml does not contain a list

The "Introduction to YAML", Part 1 lesson is supposed to discuss lists, but the list.yaml file actually contains a dictionary (YAML mapping). It makes this lesson a bit misleading.

@pklimai Good catch! This was probably the result of some copypasta in the hurried rush to get things ready for release. I know this lesson went through a few changes, so it probably just got mixed up with the dict example.
Would you like to open a PR to convert these four items into a list (add -  before each)?

Sure, will do!


A little fix and Security fix

Thank you so much for your fix. Your work is appreciated very much :D


[SwiftDate 4.x] Convert 12Hr UTC dateString into 24Hr local timezone dateString using SwiftDate

how do I convert 12Hr UTC dateString into 24Hr local timezone(IST) dateString
E.g. UTC dateString is "31-12-2018 08:30 PM" and need to convert this string into 24Hr IST timezone. (Expected result dateString in 24Hr timezone is "01-01-2019 02:00")

Hi @malcommac ,
Any updates on this issue.


details

Here's a running list of small things that need to be finished

[x] fill in placeholders for refs (@teixeirak )
[x] eliminate some references (#125 ; @teixeirak )
[ ] quilt plot methods paragraph needs some attention from @camerondow35 and dendro coauthors

quilt plot methods paragraph needs some attention from @camerondow35 and dendro coauthors

I answered the comments in the quiltplot method paragraph in my latest commit.


Generate collection UUIDs

We should generate and store collection UUIDs. They should be visible in the listCollections output.

@AlekSi I'm interested in this issue. I can contribute to this.


Make withClient function not private

Fixes #issue_number
Problem
Explain here the context, and why you're making that change.
What is the problem you're trying to solve?
Solution
Describe the modifications you've done.
Notes
Additional notes.
Checklist

[ ] Unit test all changes
[ ] Update README.md if applicable
[ ] Add [WIP] to the pull request title if it's work in progress
[ ] Squash commits that aren't meaningful changes
[ ] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

@cambridgemike there's a merge conflict, could you rebase your branch?

Sorry about that @fwbrasil looks like it's good to go now

@cambridgemike no worries!


Needs Support for Specifying Title for Notification

Some notifications require a title to be specified, like the IOTLink notifications for example,
This is a working service call from my developer-tools/service page.
service: script.notify_freddys_desktop
data:
  message: The Body of this Notification is a test too.
  title: Test Notification

This is what is required to send a notification, In order for it to work I must specify a title and a message.
But when trying to use this card and specifying a title like this
type: custom:notify-card
target: script.notify_freddys_desktop_quick
data:
  title: Quick Notification

It doesn't work, I'm assuming this is because the service call the card is trying to do is this
service: script.notify_freddys_desktop
data:
  message: Body of the Notification
  data:
    title: Quick Notification

which isn't actually specifying a title at all.
I've worked around this issue, by making a modified version of my script that specifies a title in the script, but this means I have to create a new script just for this card to use for every machine and each specific card in my UI that I may want to specify a different title for. Not ideal, but it works.
I think adding support for notify services that may require title would be worth it. My suggestion for supporting titles specifically would be to add a new option in the card's config
specify_title: true #Enables a second text field in the card for specifying the notification title at will, defaults to false
default_title: "Quick Notification" #If specify_title is false, use this title for all notifs sent with this card, if specify_title is true, then just have this be the default contents of the title field, if not specified behavior would just be identical to how the card works now.

Even in applications where the title isn't required, I think everyone using this card would benefit from being able to add a title to their Notifications

I have implemented that feature now. You can either specify a fixed title like this:
type: 'custom:notify-card'
target: living_room_tv
notification_title: 'New Notification'

or if you want to have a textfield:
type: 'custom:notify-card'
target: living_room_tv
notification_title:
  input:

If you want to change the label of the new textfield:
type: 'custom:notify-card'
target: living_room_tv
notification_title:
  input: 'Title here'


Problem serializing Response when deployed to AWS Lambda

Hi, I'm having an issue when I've deployed my Lambda to AWS and hooked it up to API Gateway.
I use the API Gateway Test functionality to test the Lambda.  It makes the call, the code executes, but when I do the Response.ok()... I get the following messages in my CloudWatch log:
org.glassfish.jersey.message.internal.WriterInterceptorExecutor$TerminalWriterInterceptor aroundWriteTo
SEVERE: MessageBodyWriter not found for media type=application/json, type=class com...
From what I've read, there's a problem with the glassfish jersey media implementation.  So I'd like to just switch over to Jackson's implementation.  Is this something you've run into or done in the past?
Thanks,
David

Hi @davidmco65

Do you have a dependency to org.glassfish.jersey.media:jersey-media-json-jackson:2.23?
Is your DTO a JAXB class (@XmlRootElement)? If so, are you sure you have a no-args constructor?
If you invoke the function through APIGateway - for example using curl - does it work?

If 1. and 2. does not apply in your case: can you send me some code + dependencies, please.

Also:
I quickly deployed https://github.com/bbilger/jrestless-examples/tree/master/aws/gateway/aws-gateway-usage-example, tested it through APIGW's test functionality and can't see any issues...

Here are my dependencies in my pom.xml file:
		<dependency> 			<groupId>com.jrestless.aws</groupId> 			<artifactId>jrestless-aws-gateway-handler</artifactId> 			<version>0.4.0</version> 			<exclusions> 				<exclusion> 					<groupId>org.glassfish.jersey.containers</groupId> 					<artifactId>project</artifactId> 				</exclusion> 			</exclusions> 		</dependency> 		<dependency> 			<groupId>org.glassfish.jersey.core</groupId> 			<artifactId>jersey-server</artifactId> 			<version>2.23</version> 		</dependency> 		<dependency> 			<groupId>org.glassfish.jersey.media</groupId> 			<artifactId>jersey-media-json-jackson</artifactId> 			<version>2.23</version> 		</dependency> 		<dependency> 			<groupId>junit</groupId> 			<artifactId>junit</artifactId> 			<version>4.12</version> 		</dependency> 		<dependency> 			<groupId>org.hibernate</groupId> 			<artifactId>hibernate-core</artifactId> 			<version>5.2.5.Final</version> 		</dependency> 		<dependency> 			<groupId>mysql</groupId> 			<artifactId>mysql-connector-java</artifactId> 			<version>5.1.40</version> 		</dependency> 		<dependency> 			<groupId>com.github.v-ladynev</groupId> 			<artifactId>fluent-hibernate-core</artifactId> 			<version>0.3.1</version> 		</dependency>
My DTO is a Pojo:
`public class ContactObject implements Serializable
{
/**
 *
 */
private static final long serialVersionUID =<PHONE_NUMBER>257579780L;
private ContactObject.Contact contact;
private ContactObject.Address address;
private String contactType;

/**
 *
 */
public ContactObject()
{
}

...

`
Do I really need the @XmlRootElement if I'm really only interested in Json response body?
I can test through API GW Test function, but can't through cURL.. just get "Missing Authentication Header" error.
Thanks for your help!
David

Maven, I see. I remember having some issues with maven and jrestless or rather jersey in an uber-jar...
Are you configuring the maven shade plugin as follows:
<configuration>
   <createDependencyReducedPom>false</createDependencyReducedPom>
  <!-- http://stackoverflow.com/questions/29107376/jersey-problems-with-maven-shade-plugin -->
  <transformers>
    <transformer implementation="org.apache.maven.plugins.shade.resource.ServicesResourceTransformer" />
    </transformers>
</configuration>

See https://github.com/bbilger/jrestless-examples/blob/master/pom.xml#L48

Sorry @davidmco65, I was short on time today.
http://stackoverflow.com/questions/29107376/jersey-problems-with-maven-shade-plugin looks exactly like your issue => try to configure maven-shade as suggested above.
You also don't need to exclude  org.glassfish.jersey.containers in jersestless 0.4.0, anymore (see #14)
You also don't need to use "@XmlRootElement" if you don't need XML (using it has the advantage that you can return both JSON and XML)
If you cannot access it via curl, then you probably didn't set up APIGW properly.
Let me know if it works.

Hi Bjorn,
Thank you for your help!  Your previous message about the configuration
of the Maven Shade plugin did the trick! Thanks so much for your
valuable library and all of the time you've obviously put into it.  It
will be the basis of all of our services going forward.
Regards,
David
On 1/20/2017 10:33 AM, Björn Bilger wrote:

Sorry @davidmco65 https://github.com/davidmco65, I was short on time
today.
http://stackoverflow.com/questions/29107376/jersey-problems-with-maven-shade-plugin
looks exactly like your issue => try to configure maven-shade as
suggested above.
You also don't need to exclude |org.glassfish.jersey.containers| in
jersestless 0.4.0, anymore (see #14
https://github.com/bbilger/jrestless/issues/14)
You also don't need to use "@XmlRootElement" if you don't need XML
(using it has the advantage that you can return both JSON and XML)
If you cannot access it via curl, then you probably didn't set up
APIGW properly.
Let me know if it works.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bbilger/jrestless/issues/26#issuecomment-274130903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQH5S4NdlMM10qIamVhdYNDHNbv-8ba2ks5rUO_WgaJpZM4LoV11.

I am happy to help!
Good to hear it's used and valuable.


emit touchstart and touchend

Better user experience for users to do linked scrolling between a single vertical FlexScrollView holding multiple horizontal FlexScrollViews. Previously using only scrollend instead of touchend results in any scrolling being blocked until scrollend (where, instead a user would expect to be able to scroll something else as soon as their finger is lifted).
Horizontal scrollers (create multiple of these):
var elem = ...a FlexScrollView;
HorizontalScrolls.push(elem);
elem.on('scrollstart', function(event) {
VerticalFlex.setOptions({ enabled: false });
elem.setOptions({ touchMoveDirectionThresshold: undefined });
});
elem.on(['scrollstop','touchend','touchcancel'], function(event) {
VerticalFlex.setOptions({ enabled: true });
elem.setOptions({ touchMoveDirectionThresshold: 0.2 });
});
Vertical scroller:
// When the vertical scrollview starts scrolling, capture all events
// and disable scrolling on the horizontal scrollview
VerticalFlex.on('scrollstart', function(event) {
HorizontalScrolls.forEach(function(hView){
hView.setOptions({ enabled: false });
});
VerticalFlex.setOptions({ touchMoveDirectionThresshold: undefined });
});
VerticalFlex.on(['scrollstop','touchend','touchcancel'], function(event) {
HorizontalScrolls.forEach(function(hView){
hView.setOptions({ enabled: true });
});
VerticalFlex.setOptions({ touchMoveDirectionThresshold: 0.5 });
});

Aweome dude! I'll have a closer look at this later this week.

Hey Nick, I've modified your suggestion a bit. Instead of emitting the 'touchstart' and  'touchend' events, I've implemented a more generic mechanism of so called 'swipe' events ('swipestart', 'swipeupdate' and 'swipeend'). These events are emitted when you use touch but also when you use the mouse (when the option 'mouseMove' is enabled). Also, they are only emmited once when you start touching and stop touching (as opposed to the default 'touchstart' and 'touchend' events, which are emitted for every finger that touches and releases the screen).
Let me know whether these events help you to improve the user experience for vertical/horizontal scrolling.


Feature/releaseapp

Add a new app to MetaCI to track Releases! This is going to start out lightweight, and grow with time as we automate the process.
The impetus behind this initial PR is to have a Release record in MetaCI that we can relate Release Test Results to for a test dashboard. When a push event is received for a production release tag (regex defined on Repo), the webhook will auto-create a Release record withe the details we can glean from the tag.
Instead of JUST being a repo and a name at this time, I've added the basic fields that we fill out on the Release Checklist which release engineering can fill out in the django admin instead. providing a relational store for this data potentially provides a better backend for statusite than the plain github releases, for example.
Additionally, we can automatically and manually associate builds with a release. Builds that are triggered from a push event that matches the release regex will be automatically connected to the release. Manual builds can select a relevant release for the current repo.
Finally, a quick update to good old Plan Run Page... Introducing: advanced mode! this is where I want the most feedback, both in implementation (I don't necessarily think this is the best way of doing it, it was just the fastest) and the actual change. I put the selector for a release behind a url parameter /plans/4/SalesforceFoundation/Cumulus/run/?advanced_mode=1. I also put the Commit field hidden unless specify the parameter... Is a URL Param an okay for now way of doing this? Is the implementation of adding to the form layout reasonable? Should commit also be hidden?
This is not yet a prescriptive design for release automation in MetaCI. stay tuned...

:bug: "Builds that are triggered from a push event that matches the release regex will be automatically connected to the release." doesn't work quite yet...

Ready rereview @davisagli


v2.1

Coverage decreased (-1.09%) to 96.312% when pulling f0b1288ca6d40c461df6b30296d77b9ac5425371 on development into 23f21756562adbd04c172d14995802030bf81595 on master.

Coverage decreased (-1.09%) to 96.312% when pulling f0b1288ca6d40c461df6b30296d77b9ac5425371 on development into 23f21756562adbd04c172d14995802030bf81595 on master.

Coverage decreased (-1.09%) to 96.312% when pulling f0b1288ca6d40c461df6b30296d77b9ac5425371 on development into 23f21756562adbd04c172d14995802030bf81595 on master.


Avoid using paste to combine multiple link lists.

Current caveman method using paste to combine multiple link lists is not safe. Instead append individual strings, using conditionals to prevent failures when any of the mirrors is down, or any of the individual links is empty.

Fixed by c2b1e55. Don't need to skip using paste, just echo empty strings when there is no 'get' page, and paste will concatenate those empty strings (from one mirror) with the link of the other mirror.


Istio traffic routing tabs disappeared from UI in master

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Enable Istio in Rancher master
Navigate to Resources > Istio

Result:
There should be 5 tabs across the top of that page. It SHOULD look like:

But what we actually see is:

In master, I do not see "Gateways", "Destination Rules", or "Virtual Services"
Other details that may be helpful:
First screenshot above is the correct view from Rancher v2.3.5. Changing ui-index in Rancher v2.3.5 to be the same as master (https://releases.rancher.com/ui/latest2/index.html) does NOT reproduce this issue.
Environment information
Single node install, master Rancher version 580ab5bb4

Cluster information
3 nodes, all roles, 8gb 4vcpu each. Hosted on Linode.
Kubernetes Version: v1.17.2

Interesting to note: using the URLs directly takes me to the appropriate pages:
https://${RANCHER_SERVER_URL}/p/${CLUSTER:PROJECT IDs}/istio/project-istio/gateways
https://${RANCHER_SERVER_URL}/p/${CLUSTER:PROJECT IDs}/istio/project-istio/destination-rules
https://${RANCHER_SERVER_URL}/p/${CLUSTER:PROJECT IDs}/istio/project-istio/virtual-services

The display of those tabs are now behind a feature flag

The code was committed in october https://github.com/rancher/ui/commit/5562d5f65d12ee2c65b944bc34ea5b3109179410
I don't think this is in error.

Enabling that feature flag does resolve this issue. Has this specific flag been changed in master to be disabled by default now? I see it disabled even on current master (Rancher version e11d47814), but it is enabled by default using v2.3.6-rc1. I think proper behavior this one is to be enabled by default.

I don't really have an opinion on whether or not it should be enabled by default. If we want to change the default value this is going to be a backend change. The front end only checks to see if the feature is enabled, it doesn't set the default.
@maggieliu if we want to change the default the backend should change it.

If we need the flag enabled to show the UI tabs correctly then I think we should just enable the flag by default. It doesn't make sense to me to keep it disabled unless there is a very very good reason and the UI can help the user enable it themselves (would need a use case as to not enable it by default)

investigating rn, as the default should be true

Validated on Rancher version dad58367e (dad58367e)
Left ui-index and system-charts as the defaults.
The routing tabs are back! The feature flags now have the correct default values (True for istio-virtual-service-ui and False for steve and unsupported-storage-drivers. None are null anymore).
Feature defaults:

Istio traffic routing screens:


Transition duration

Hello!
I didn't find an option to somehow modify the transition effect between steps?
Specifically, I wanted to know if there is a way to make a transition "smoother".
Thank you

Hello!
Are you talking about scroll effect or CSS transition at step display?

CSS transition at step display

You can simply customize the steps template and use Vue Transitions to make it behave as you want.

Thank you, this seems to solve my problem


Initial commit with logger tool to dump all requests and responses

This tool could require some work. For example, writing the log will fail if the location_info and initialize responses/requests are not added properly. This can be fixed but will take more time. For now, if the example is followed, it will write a JSON log that can be read.

I've changed the logger so that it collects all requests and responses automatically provided a path environment variable is set to an existing directory.

Since this is a new feature, i bumped up the version string for cpp.


When using the url to pdf function, the image cannot be saved.

When using the url to pdf function, the image cannot be saved.

https://xz.aliyun.com/t/14263?time__1311=mqmx9QiQi%3DDtWq05DI74CT%2BCodunxGIdx&alichlgref=https%3A%2F%2Fxz.aliyun.com%2F

@Le1a Your images are not visible in the url itself ! Can you confirm it once ?


Showcase Submission for physics-libraries/box2d/12-collision-events

Thank you Brendan O'Donnell for your contribution! A member of the Coding Train team will review it shortly.

Projection Mapping with Collision Events.
Brendan O'Donnell

This is so fun!!! Thank you for sharing.


fix: 修复自定义列宽对虚拟数值列不生效

👀 PR includes

🐛 Bugfix

[x] Solve the issue and close #0

📝 Description

// dataCfg

  "fields": {
    "rows": ["province", "city"],
    "values": ["number"],
  },

// options

  style: {
    colCell: {
      widthByField: {
        number: 100,
      },
    }
  },

🖼 Screenshot

Before
After

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.81%. Comparing base (ecdc9c8) to head (291420d).
Report is 445 commits behind head on next.

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #2921      +/-   ##
==========================================
+ Coverage   75.77%   80.81%   +5.04%
==========================================
  Files         257      180      -77
  Lines       11994    10425    -1569
  Branches     2464     2362     -102
==========================================
- Hits         9088     8425     -663
+ Misses       1398      555     -843
+ Partials     1508     1445      -63

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:tada: This PR is included in version @antv/s2-v2.0.0-next.29 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v2.0.0-next.28 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


RNV and pre-commit: android version change doesn't get into commit

I'm using npm pre-commit package to have my build version bumped before every commit (This is the requirement and I really need this like so). For some reason RNV changes build.gradle after commit, so the version change doesn't get into that commit.
Here is my package.json:
{
  "scripts": {
    "bumb_internal_build": "node ./scripts/bumpInternalBuildNumber.js && git add ./src/buildNumber.json",
    "bump_build_android": "node ./node_modules/react-native-version/cli.js --never-amend --increment-build --target android",
  },
  "pre-commit": [
    "bumb_internal_build",
    "bump_build_android"
  ],
  "dependencies": {
    "react": "^16.6.3",
    "react-native": "^0.57.8"
  },
  "devDependencies": {
    "pre-commit": "^1.2.2"
  }
}

Am I overlooking anything?

I think you have to stage the files same as you do with git add ./src/buildNumber.json in the first script.


403 on private channels

public channels are working fine, but when I try to use Echo.private, echo server console gives me:
ghY92USRaTt7bN62AAAB could not be authenticated to private-9

<title>403 Forbidden</title>

403 Forbidden

nginx/1.10.0 (Ubuntu)

Client can not be authenticated, got HTTP status 403

I have broadcastserviceprovider set up with:
Broadcast::routes();
require base_path('routes/channels.php');

And channels.php:
Broadcast::channel('*', function ($user, $id) {
    return true;
});

I'm wondering could this somehow being related to Broadcast routes though? I'm finding nothing in logs to help me out with this, laravel-echo-server.json:
 "authHost": "http://mydomain.com",
        "authEndpoint": "/broadcasting/auth",
        "clients": [
                {
                        "appId": "idofmine",
                        "key": "keyofmine"
                }
        ],
        "database": "redis",
        "databaseConfig": {
                "redis": {},
                "sqlite": {
                        "databasePath": "/database/laravel-echo-server.sqlite"
                }
        },
        "devMode": true,
        "host": "mydomain.com",
        "port": "6001",
        "protocol": "redis",
        "socketio": {},
        "sslCertPath": "",
        "sslKeyPath": ""

Can I see your client Echo codes? Probably you are not set auth options at startup. Here is an example:
        let echo = this.echo = new Echo({
            broadcaster: 'socket.io',
            host: HOST_FOR_YOUR_SERVER,
            auth: {
                headers: {
                    Authorization: 'Bearer ' + AUTH_API_TOKEN,
                },
            },
        });

And you should change your BroadcastServiceProvider:
        Broadcast::routes(['middleware' => ['auth:api']]);

@tinpont
True I haven't done that. I got it such as:
window.Echo = new Echo({
broadcaster: 'socket.io',
host: 'http://mydomain.com:6001'
});
About the auth headers, what is that 'Bearer' there? Where I have to define that one?
And thanks for the help!

This is the document tell you how to setup a api base auth.
Which is using this class to retrieving the user. So you AUTH_API_TOKEN is your users table api_token field's value.

laravel/echo emit subscribe & unsubscribe event with option.auth data.

    createClass(SocketIoChannel, [{
        key: 'subscribe',
        value: function subscribe() {
            this.socket.emit('subscribe', {
                channel: this.name,
                auth: this.options.auth || {}
            });
        }
    }, {
        key: 'unsubscribe',
        value: function unsubscribe() {
            this.unbind();
            this.socket.emit('unsubscribe', {
                channel: this.name,
                auth: this.options.auth || {}
            });
        }
    }

laravel-echo-server receiving event.

    /**
     * On subscribe to a channel.
     *
     * @param  {object} socket
     * @return {void}
     */
    onSubscribe(socket: any): void {
        socket.on('subscribe', data => {
            this.channel.join(socket, data);
        });
    }

laravel-echo-server common channel join in.

    /**
     * Join a channel.
     *
     * @param  {object} socket
     * @param  {object} data
     * @return {void}
     */
    join(socket, data): void {
        if (data.channel) {
            if (this.isPrivate(data.channel)) {
                this.joinPrivate(socket, data);
            } else {
                socket.join(data.channel);
                this.onJoin(socket, data.channel);
            }
        }
    }

    /**
     * Join private channel, emit data to presence channels.
     *
     * @param  {object} socket
     * @param  {object} data
     * @return {void}
     */
    joinPrivate(socket: any, data: any): void {
        this.private.authenticate(socket, data).then(res => {
            socket.join(data.channel);

            if (this.isPresence(data.channel)) {
                var member = res.channel_data;
                try {
                    member = JSON.parse(res.channel_data);
                } catch (e) { }

                this.presence.join(socket, data.channel, member);
            }

            this.onJoin(socket, data.channel);
        }, error => {
            Log.error(error.reason);

            this.io.sockets.to(socket.id)
                .emit('subscription_error', data.channel, error.status);
        });
    }

laravel-echo-server private channel authenticate.

    /**
     * Send authentication request to application server.
     *
     * @param  {object} socket
     * @param  {object} data
     * @return {Promise<any>}
     */
    authenticate(socket: any, data: any): Promise<any> {
        let options = {
            url: this.authHost() + this.options.authEndpoint,
            form: { channel_name: data.channel },
            headers: (data.auth && data.auth.headers) ? data.auth.headers : {},
            rejectUnauthorized: false
        };

        return this.serverRequest(socket, options);
    }

These code explain why setup auth.headers in new Echo(options).

Many thanks @tinpont! That helps a ton :).

Looks like this was a setup issue. Closing

@tlaverdure actually that did not solve it, but it's possible it's just a setup issue in any case. I'll try to find a solution

I had a very similar issue, and I managed to solve it.
I submitted a patch upstream and am hoping it gets accepted.
Perhaps try the change out and see if it helps.
Was extremely frustrating.

@tinpont thanks  Broadcast::routes(['middleware' => ['auth:api']]); worked for me :) :+1:

Soultion to Common Problems faced by many users while setting up Laravel Echo
https://gist.github.com/rauschmerscen/2f2265976d59267f3bfccb339b27be44

i set localhost and ip and APP_URL and host to <IP_ADDRESS> thhen i send CSRF token
some time may thay have diffrent destination that why i set them in one (<IP_ADDRESS>) and it works !
and csrf like this

    window.Echo = new MyEcho({
        broadcaster: 'socket.io',
        host: '<IP_ADDRESS>:6001', // this is laravel-echo-server host
        auth:{
            headers: {
            'X-CSRF-TOKEN': '{{csrf_token()}}'
        }}

    });


Update email alert presenter tags

Change the tags to be keyed on 'policies' rather than 'policy'. This brings
email alerts in line with the rest of gov.uk in the naming of this type of
field and ensures that subscriptions are created correctly by the email-
alerts-frontend.

:+1:

Changing approach here - closing PR.


Fix for statsd not starting on first deploy.

We have experienced issues on tc where statsd exits 1 when first spinning up a cluster. This was due to a restart and a start command hitting upstart at the same time.
I removed the explicit start, since it writes the template on first deploy, it starts fine with the restart trigger.
@klobucar @tcahill @schancel

NOTE: I have done 3 deploys and not seen the issue. I will continue to test.

LGTM

Thanks for the quick review. =)


False positive on InvalidPackageDeclaration in Multiplatform project

Running gradle :sample:app:detektMetadataMain on https://github.com/eygraber/compose-color-picker results in 2 InvalidPackageDeclaration issues, but the package declaration seems fine to me.

Also not sure if this is related or not, but it seems like the excludes property in the config isn't working for it. I have the following in the config:
  InvalidPackageDeclaration:
    active: true
    excludes: ['*.kts']
    rootPackage: ''

but if I run detekt on a build.gradle.kts file, it causes a InvalidPackageDeclaration report.

Also not sure if this is related or not, but it seems like the excludes property in the config isn't working for it. I have the following in the config:

I don't think it is. Anyway, you should probably use excludes: ['**/*.kts'] to exclude the
build.gradle.kts file.

Anyway, you should probably use excludes: ['**/*.kts'] to exclude the
build.gradle.kts file.

I think what I'm using is what was generated by the task that makes a config for you.

Using ['**/*.kts'] worked.

is what was generated by the task that makes a config for you.

Yup seems like we have a bug there.

Was this the root cause of your false positive @eygraber ?

No, that's still happening.

No, that's still happening.

Can you provide what's the snippet that is causing the issue?

It's happening on both of the files in this package https://github.com/eygraber/compose-color-picker/tree/master/sample/app/src/commonMain/kotlin/com.eygraber.compose.colorpicker.sample

The reason is fairly simple. The file Sample.kt is expected to be in directory
./compose-color-picker/sample/app/src/commonMain/kotlin/com/eygraber/compose/colorpicker/sample/Sample.kt

but is found in
./compose-color-picker/sample/app/src/commonMain/kotlin/com.eygraber.compose.colorpicker.sample/Sample.kt

Well that explains that :see_no_evil:


KIC doesn't fail properly when a Service referred to by an Ingress is not available in the storer cache when running the Kong configuration parser

While testing in next I found that missing K8sServices in the storer cache can result in broken kong.Services which will be deployed in a broken state to the Kong Admin API.
The purpose of this issue is to cause a hard failure instead of a logged error but continuing with the POST update in these cases.

Today, if this happens, KIC will try to push corrupted state into Kong.
Possible implementations:

if KIC detects a missing service in its Storer cache, fail loudly and prevent a faulty sync from happening
if KIC detects a missing service in its Storer cache, try to repair corrupted internal state by pulling the missing service from the k8s api

Due to the theoretical nature of this issue we don't have priority for it at the moment so we'll close it for the time being. If anyone starts running into this issue please feel free to comment here and we'll be happy to re-open it and re-assess.


Un-focus nav when a page nav link is clicked (on mobile)

There's really no reason to not do this as it's expected that somebody navigating to a new page wants to interact with that page immediately.
Either we can do a page refresh like on some other links / buttons on the site or we can trigger the "close the nav" js.

added


[BUG]

Describe the bug
Stuck at Locating gate way
To Reproduce
Steps to reproduce the behavior:

open Beacon.exe
2 get stuck ON Locating gateway
3.?????
profit

Expected behavior
to work?
Screenshots
no need i hope
Environment (please complete the following information):

OS: Windows 10 version 2004
Application Version 2.4.0

Additional context
getting crash report
https://pastebin.com/nqxZsEXd

This issue duplicates #10 which has been submitted via the error reporter sometime yesterday morning.
As it stands, this issue occurs when no compatible router is found within your network within the time limit of 5 Minutes (this timeout duration has since been lowered significantly). While I agree that it needs to be fixed (thus the existence of #10), it still means that your router most likely does not have the necessary prerequisites to work with Beacon (either UPnP has been disabled or the functionality simply isn't supported at all).
Going by the announcements in your log, you seem to be using the Tilgin HG2280 which does not seem to mention any support for port forwarding via UPnP in any of the documentation I could find. You may have some luck in the administration interface, however.


Set RES_EMBED_USE_SHARED for add_subdirectory usage

This variable is used for determining linkage type in ResEmbed function and it is defined in res_embedConfig.cmake which is utilized when res embed is installed as a package in the system. In order to use resembed by add_subdirectory we need to set this variable and for it to take effect we need to set scope to parent scope.

Hi @NCirit , I have a feeling now that RES_EMBED_USE_SHARED is redundant, because the availability of res::embed or res:embed::static can be checked directly by doing if (TARGET res::embed) and so on. If this is the only reason why RES_EMBED_USE_SHARED is required, it should not be used. I've already replaced one internal use of RES_EMBED_USE_SHARED, which is clearly redundant here: fc2ec263ae2160ca5e495617494334aeae37623c . What do you think?

Hi @dmikushin, just checked the code and yes it seems redundant now.

Thanks, @NCirit ! Following your confirmation, I remove it it 382f4b823ff66c3352690d65c34d0ca02c8b4d8a