Fix headers in Ollama requests

I've noticed there's work done regarding passing headers during initializing Ollama, but it's not complete/ not working well. I've fixed the code. Please check it :)

Example usage:
const ollama = new Ollama({
  host: "https://ollama.host.com",
  headers: {
    'Authorization': 'Basic dXNlcm5hbWU6cGFzc3dvcmQ=', // username:password base64 encoded
  },
});


Screencast request fails with Other

This is just running the demo, same thing happens if I try to actually write/use code, the error is throne at the
let session = proxy.create_session();

for when trying to do this in code. Error: Portal request didn't succeed: Other

Something must be happening from the portal itself, not ashpd.

Closing as this is probably an issue on the xdg-desktop-portal implementation you are using


PostgreSQL - DDL generation for sequences does not work

Hi,
DBeaver 3.7.5
PostgreSQL 9.5.4
I try to get the DDL of a sequence with a left clic to the sequence in the database navigator, then "Generate SQL", then "DDL".
I get a NullPointerException.
I suppose this is not supported yet, isn't it?
Regards,
Cedric

Yep.
But that's easy to do, I'll add it in the next version.

Thanks for reactivity.


link the binary to the cargo bin directory when building from source

Normally ~/.cargo/bin is in your PATH so symlinking the file there makes it easily executable when building it from source.

This is what happens automatically when you cargo install --path cli in the repo anyways.. soo...

okay so should the install command be modified then? for sure the better solution


Add encoding=utf-8 when opening a file

Without this flag, the package can't be installed (setup.py) or run on Windows.
Because Python will open a file by GBK codec on Windows if there it is not set to utf-8.

already merged, thank you!


DynDNS: get_dyndns_ip() Chinese alternative capture regex

Alternative host also requires an alternative capture regex.

currently seems to do the same as it already did, closing this for now.

Um no.  It doesn't do the same as before.  It fixes what was before.  Changing only the host does not work because the current capture regex does not match.  The capture regex also need to be changed.


Update docs to show intermediates with client auth example

In https://github.com/square/okhttp/blob/ef5d0c83f7bbd3a0c0534e7ca23cbc4ee7550f3b/okhttp-tls/README.md
Relates to https://stackoverflow.com/questions/67064309/bad-certificate-error-when-using-client-certificate-with-retrofit-okhhtp-but-w/67087544#67087544

Closing as low priority


Borders example is broken

Bevy version
main: commit 956d9ccbb1e6b240f3e434acaab9ad544df41102
Relevant system information
AdapterInfo { name: "Intel(R) UHD Graphics 620 (KBL GT2)", vendor: 32902, device: 22807, device_type: IntegratedGpu, driver: "Intel open-source Mesa driver", driver_info: "Mesa 24.1.6-arch1.1", backend: Vulkan }

What you did
cargo run --example borders

What went wrong
It's supposed to look like this:

But the rounded borders flicker out of existence immediately after loading:

Additional information
I have a strong suspicion that this has to do with https://github.com/bevyengine/bevy/pull/15163, which was merged 5 days ago, so I'm pinging @ickshonpe. The fact that it's very briefly visible also makes me think that this might be a TemporaryRenderEntity while it is supposed to persist. I'm not sure however as I believe I've tested this before merging #15320, and it was also broken.
Actually I'm 90% sure that it's a TemporaryRenderEntity while it isn't supposed to be.

I can't reproduce, it looks fine for me. I could get the bottom row to disappear if I resize my Window to be too small, but they reappear when I resize it back.
SystemInfo { os: "Linux 24.05 NixOS", kernel: "6.6.46", cpu: "AMD Ryzen 7 5800X 8-Core Processor", core_count: "8", memory: "47.0 GiB" }
AdapterInfo { name: "NVIDIA GeForce RTX 3070", vendor: 4318, device: 9352, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "550.78", backend: Vulkan }

Yes it looks fine to me too, I'm not sure why just the second set of nodes would dissappear like that.

@Trashtalk217 does anything else disappear/reappear if you change the window size?

I can't reproduce this on my Windows laptop with an integrated GPU.

2024-10-01T22:05:34.349941Z  INFO bevy_render::renderer: AdapterInfo { name: "Intel(R) Iris(R) Xe Graphics", vendor: 32902, device: 39497, device_type: IntegratedGpu, driver: "Intel Corporation", driver_info: "101.5768", backend: Vulkan }

Moving and resizing the window works fine.


Starting non node applications, fails script does not exist

I am trying to use forever to run non node applications.
This the command I use
$ forever start -c nc -l 53351
Forever exits with following the message.
error:  Cannot start forever
error:  script /home/tydall/start does not exist.
I am going about this the wrong way? or should this work ?

forever is always looking explicity in the current working directory for the script you specify.
As a workaround:
 forever start `which nc` -l 53351

The command, exactly like that with backticks, should put the output of which nc into the command, which, being an absolute path, should work.  (As should just using the absolute path of the binary in the first place).

Even if I give the fullpath to nc the result is the same.
Looking at the output from forever it looks like its trying to run a script called "start"


after 0.4 hackage update, tons of documentation is missing

This is extremely unfortunate due to the fact that the documentation was very complete and probably among your best.

With the exception of one example code block, all the documentation is still there. I suspect that the problem is that hackage currently uses an old version of hackage that fails to present haddocks attached to types and functions reexported from internal libraries. If you clone sockets from github and run cabal v2-build && cabal v2-haddock (with cabal-install-3.0 at least, not sure about 2.4), you can see all documentation shows up. I'll manually upload the documentation when I release sockets-0.6.

still can't see the doc on hackage :(

The main thing preventing this from working at the moment is that posix-api depends on the systemd library being available, so hackage fails to build it. If that were fixed, then a new release of sockets would get its docs built by hackage.

And the fix for the posix-api problem would be doing some kind of shim (just crashing) on systems that do not have the systemd library available.


Make confvals linkable again

Feature request
Make confvals linkable again

That also means, that a confval must be unique (to determine the link destination) and duplicate confval definitions must output a warning (like in Sphinx). This is currently not the case.

I am working on some idea


High_voltage and Pundit, only auth some pages?

I want to use high_voltage for most pages, but then have only a couple that require a user to be logged in to view.  Tried following the README for it, but ran in to a mixture of issues, probably compounded by me trying various things.
What I want to have is:

root route set to / instead of /pages
/help and /contact require a user to be logged in
/ just displayed as usual

I am using Pundit and have it set to require authorization everywhere.  Is it possible to do this and still use high_voltage?

@rchady I am not familiar with Pundit however, you could potentially resolve this by overriding the pages controller as documented in https://github.com/thoughtbot/high_voltage#override to conditionally run authentication on some pages and not others.


Enhance 3 remaining rules (942521, 943110, 943120) where affected parameter is not visible in alert message

@airween has improved quite a few of these rules. See #3409 for a pattern how to do this.
Here are example the alert messages if the 3 rules in question:
## 942521
[2023-12-15 09:39:53.898812] [security2:error] <IP_ADDRESS>:37032 ZXwQ2fUpwlBSgEDtAzsRagAAABE [client <IP_ADDRESS>] ModSecurity: Warning. Pattern match "^(?:and|or)$" at TX:1. [file "/home/dune73/crs/rules/REQUEST-942-APPLICATION-ATTACK-SQLI.conf"] [line "1536"] [id "942521"] [msg "Detects basic SQL authentication bypass attempts 4.1/4"] [data "Matched Data: -1839' or found within TX:1: or"] [severity "CRITICAL"] [ver "OWASP_CRS/4.0.0-rc2"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-sqli"] [tag "OWASP_CRS"] [tag "capec/1000/152/248/66"] [tag "PCI/6.5.2"] [tag "paranoia-level/2"] [hostname "localhost"] [uri "/"] [unique_id "ZXwQ2fUpwlBSgEDtAzsRagAAABE"]
## 943110
[2023-12-15 09:39:58.868778] [security2:error] <IP_ADDRESS>:58950 ZXwQ3vUpwlBSgEDtAzsa3AAAABA [client <IP_ADDRESS>] ModSecurity: Warning. Match of "endsWith %{request_headers.host}" against "TX:1" required. [file "/home/dune73/crs/rules/REQUEST-943-APPLICATION-ATTACK-SESSION-FIXATION.conf"] [line "69"] [id "943110"] [msg "Possible Session Fixation Attack: SessionID Parameter Name with Off-Domain Referer"] [data "Matched Data: http://www.attackersite.com/ found within TX:1: www.attackersite.com"] [severity "CRITICAL"] [ver "OWASP_CRS/4.0.0-rc2"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-fixation"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/225/21/593/61"] [hostname "localhost"] [uri "/login.php"] [unique_id "ZXwQ3vUpwlBSgEDtAzsa3AAAABA"]
## 943120
[2023-12-15 09:39:58.890206] [security2:error] <IP_ADDRESS>:59046 ZXwQ3vUpwlBSgEDtAzsa6AAAABQ [client <IP_ADDRESS>] ModSecurity: Warning. Operator EQ matched 0 at REQUEST_HEADERS. [file "/home/dune73/crs/rules/REQUEST-943-APPLICATION-ATTACK-SESSION-FIXATION.conf"] [line "95"] [id "943120"] [msg "Possible Session Fixation Attack: SessionID Parameter Name with No Referer"] [data "Matched Data: jsessionid found within REQUEST_HEADERS: 0"] [severity "CRITICAL"] [ver "OWASP_CRS/4.0.0-rc2"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-fixation"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/225/21/593/61"] [hostname "localhost"] [uri "/login.php"] [unique_id "ZXwQ3vUpwlBSgEDtAzsa6AAAABQ"]

This is a problem that only affects chained rules. With the 92x rules, it's usually clear what's at stake even if the log message does not reveal the original parameter hit. (-> because the rule inspects only the Accept header for example). But with the application rules, you are left in the dark. So the log messages of these rules should be enhanced.
I went through all the chained rules systematically and checked the log message. These seem to be the only remaining ones where we need to update the rule.
I do not think this is mandatory for a v4 release, but it would be sweet.
Also: I did this to make sure my C-Rex software is able to pick the correct parameter for every CRS rule alert, always. Catching up on a few of the rules now.

Thanks, I'm going to take a look this soon.
Btw. I think more and more that the mentioned check should be integrated into our pipeline.

Thank you

I'm wondering why 942440 does not follows this behavior:
[id "942440"] [msg "SQL Comment Sequence Detected"] [data "Matched Data: ;-- found within ARGS:var: DROP sampletable;--"]

This rule uses MATCHED_VARS (there is an S at the end) and not MATCHED_VAR.
May be we can use this solution for each cases?

Rule 931131 also uses an interesting solution which avoids this behavior.

Yes, I saw the 931131 too. Maybe we can converge on a single variant and not like every chained rule re-invents the wheel.


Wrong HEVC video orientation

When playing VoD mp4 input with HEVC codec and portrait orientation, the stream is rotated 90 degrees. Although there are no problems with landscape orientation.

Hey there!
It sounds like your video is stored as a landscape-orientation video, being forced to play in portrait orientation during decode through MP4 metadata flags.
We consider this "working as intended", as this kind of orientation information cannot be reliably converted between different container formats (many containers simply don't have this concept at all), so MistServer doesn't even attempt to process/output these flags.
You can, however, play videos that are truly in portrait orientation normally as you'd expect (re-encoded in portrait orientation, as opposed to being rotated client-side during playback).
Hope that helps!


Update master branch name to main

Update master branch name to main. This is standard now across github and should be consistent across our projects. Thoughts?

@redhatrises done here, but if there are other repos that need it, lets def do it


Assets are never re-compiled

There seems to be a caching issue happening—assets:precompile does nothing even if new classes are used and purging needs to happen again. I have to run assets:clobber first to force a full re-compile.

Just so I understand, based on https://github.com/rails/tailwindcss-rails/blob/main/lib/tailwindcss/engine.rb
The tailwind file is registered, but that file is never going to change, right? I assume Sprockets will never trigger a new compile if the source files don't change... is there a way to tell Sprockets that the compressor is going to produce different output this time, so it should re-compile even if the inputs are the same?

Just so I understand, based on https://github.com/rails/tailwindcss-rails/blob/main/lib/tailwindcss/engine.rb
The tailwind file is registered, but that file is never going to change, right? I assume Sprockets will never trigger a new compile if the source files don't change... is there a way to tell Sprockets that the compressor is going to produce different output this time, so it should re-compile even if the inputs are the same?

Yeah, we should document this limitation at the moment. Feel free to look into whether there's a way to tell sprockets that the compressor thinks its time for a rebundle, even if the source didn't change.

Yeah, we should document this limitation at the moment. Feel free to look into whether there's a way to tell sprockets that the compressor thinks its time for a rebundle, even if the source didn't change.

Sprockets has a depend_on directive that would work, if we pass in the full list of all the files that have been considered for class lookups. Will try to raise a PR using this method, unless someone has a better way to handle this.

Sprockets has a depend_on directive that would work, if we pass in the full list of all the files that have been considered for class lookups. Will try to raise a PR using this method, unless someone has a better way to handle this.

Oh, that's a great idea. Let's give that a try 👍

Oh, that's a great idea. Let's give that a try 👍

PR #12 raised, uses the dependencies key in the output and passes the digest URI for all the files that we extract classnames from.

PR #12 raised, uses the dependencies key in the output and passes the digest URI for all the files that we extract classnames from.

Documented need for clobber in a2abc429a241d44be713d775448b246bf0cec534.

Documented need for clobber in a2abc429a241d44be713d775448b246bf0cec534.

The way I'm working around this now is to disable the asset cache using the instructions in https://guides.rubyonrails.org/v6.1/asset_pipeline.html#assets-cache-store
I'd suggest documenting this as the recommended approach, it's the only way I can see that handles all the edge cases.
It also works to automatically fix the Heroku build even if it was deployed without switching off the cache.

The way I'm working around this now is to disable the asset cache using the instructions in https://guides.rubyonrails.org/v6.1/asset_pipeline.html#assets-cache-store
I'd suggest documenting this as the recommended approach, it's the only way I can see that handles all the edge cases.
It also works to automatically fix the Heroku build even if it was deployed without switching off the cache.

Made this the default in 2097fc4af21a6ddfc62e73afa391398e498ab7be.

Made this the default in 2097fc4af21a6ddfc62e73afa391398e498ab7be.


Get-DbaRegServer -Debug:$false Triggers Debug Messages

I'm using $DebugPreference = 'Continue' to debug my script.  I can't suppress debug messages from the cmdlets called by Get-DbaRegServer.  It might be something with how Write-Message is used instead of Write-Debug.  This issue might not be isolated to just these cmdlets.  It likely to applies to verbose and information level messages.  The issue applies to both 5.1 and 7.1 versions of PowerShell.
$DebugPreference = 'SilentlyContinue'
Get-DbaRegServer -SqlInstance 'cmsserver' -Group 'DISABLED', 'IGNORE'
<#
No debug messages as expected
#>
$DebugPreference = 'SilentlyContinue'
Get-DbaRegServer -SqlInstance 'cmsserver' -Group 'DISABLED', 'IGNORE' -Debug:$false
<#
Unexpected debug messages
DEBUG: 2274 | [15:05:55][Connect-DbaInstance] Starting process block
...
DEBUG: 34152 | [15:05:56][Get-DbaRegServerGroup] Processing IGNORE
#>
$DebugPreference = 'Continue'
Get-DbaRegServer -SqlInstance 'cmsserver' -Group 'DISABLED', 'IGNORE'
<#
Debug messages as expected
#>
$DebugPreference = 'Continue'
Get-DbaRegServer -SqlInstance 'cmsserver' -Group 'DISABLED', 'IGNORE' -Debug:$false
<#
Unexpected debug messages
DEBUG: 2274 | [15:05:55][Connect-DbaInstance] Starting process block
...
DEBUG: 34152 | [15:05:56][Get-DbaRegServerGroup] Processing IGNORE
#>

Please use the proper issue template.
If you do not want the debug messages to show up do not specify the debug parameter and you will not see them.

With cmdletbinding, it is legal to set the value of a common parameter switch.  For example, "-debug", "-debug:$true", and "-debug:$false" are all legal syntax.  However, these are not treated correctly.
I normally debug only my script.  I use "-debug:$false" on a 3rd party cmdlets unless I am specifically debugging the 3rd party cmdlet.  If coded properly, a cmdlet will honor the value passed.  As it is now, I can't use the common parameters because of the bug...or feature.  As a work around, I have to set the preferences just before and after each call to DBATools cmdlets if I want to control the preferences.
It is an interesting and old topic.  I wish I had more time to explore it.  There is a twist for modules.
https://devblogs.microsoft.com/scripting/weekend-scripter-access-powershell-preference-variables/
https://stackoverflow.com/questions/4301562/how-to-properly-use-the-verbose-and-debug-parameters-in-a-custom-cmdlet/20830886#20830886
https://developer42.wordpress.com/2017/02/04/powershell-suggestion-simplify-write-verbose-in-modules/
https://stackoverflow.com/questions/2775357/how-does-a-cmdlet-know-when-it-really-should-call-writeverbose/20828118#20828118
BTW, when I clicked on new issue, I did not get a template, so perhaps I'm doing this incorrectly.

FYI  I just clicked new issue again and got the template list.  I have no idea how it got skipped the last time.

That is part of the logging system we have in the module. You can create a bug report if you would like for this but not sure how fast we will be able to address it.

I will create a bug report.  If the logging is in one place, then at least it's not a function by function fix.  Thanks.


Issues with stand-alone .http file

I was doing a demo trying to demonstrate simplicity of httpYac and ran into a weird issue. So, I was trying to demonstrate the use of a single, standalone .http file (not a folder). It loads the file fine and I can run the tests using the CodeLens options, but when I try to open the Testing view or jump to it via the Reveal test in Explorer context menu, the test buttons in the source window disappear and no tests get displayed in the Testing window.
https://github.com/user-attachments/assets/8652bd49-9f0e-41b0-9f60-01ee2cbe7f08
But there is a split second when the Testing window gets to display the tests before it switches to the message that additional extensions may be needed.

I also notice that when I try to load the same file (doing pretty much the same), sometimes the Testing menu would appear fine.

When I open the Test Runner, I first load all the test files. I use the entire current workspace for this. In this scenario, however, no workspace is open and previously discovered test cases are overwritten. I now use all visible files as a fallback in such a case.


High skip ingest fix

Codecov Report

Merging #384 (8b3c859) into master (7a63c01) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #384      +/-   ##
==========================================
+ Coverage   41.43%   41.49%   +0.05%
==========================================
  Files          32       32
  Lines        4817     4813       -4
==========================================
+ Hits         1996     1997       +1
+ Misses       2821     2816       -5

Impacted Files
Coverage Δ

assemblyline_core/ingester/ingester.py
76.69% <100.00%> (+0.55%)
:arrow_up:

assemblyline_core/dispatching/dispatcher.py
76.86% <0.00%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

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


codec(ticdc): avro simplify the unit test

What problem does this PR solve?

Issue Number: close #11014
What is changed and how it works?

remove some useless avro unit test
use schema test helper to generate the events

Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`

/retest

/retest

/retest

/retest

/retest

/retest


Update streaming-programming-guide.md

Update See the Scala example to See the Java example.

Can one of the admins verify this patch?

@yosssi these are OK as minor changes that don't need a JIRA, but "Update streaming-programming-guide.md" is not sufficiently descriptive as a title. Anything more specific would be helpful, like "Fix Scala/Java example typo in streaming programming guide"

@srowen Thanks for your advice. I modified this issue's title.


Some methods appear in xml files, but not in the editor

Godot version:
3.2 beta6
Issue description:
For example the get_patch_margin() in NinePatchRect. It has an entry in the xml file of the class, but you can't find it in the editor help. This probably applies to all getters/setters that use a default enum argument.

It does appear, as the getter of the four patch_margin_* properties.
But yes, composite setters with 2 arguments or getters with 1 argument are a bit unconventional and are not really documented in the best way - but they're also not really meant to be used, their implementation is a backend detail.

It does appear, as the getter of the four patch_margin_* properties.

But you can also write the description (unlike other setters/getters) and it's nowhere to be seen. That's the main problem.

Oh I see. That's a bug in the editor indeed.
We chose to expose those in the classref explicitly since their documentation is a bit more than just "sets ", as what property is set depends on an enum value. See #30125.
Seems like I did not check that they'd actually appear in the editor, there might be some code to filter them out.


Update for 2.0

This makes it work in 2.0, and adds checkboxes for unused resources.
I'm not satisfied with the UI layout, but at least it works. Anyone with more time to rearrange UI can make it better. It still needs room for multiple types of cliffs.
I haven't done a PR in 2 years so I expect I'm doing something wrong here, with making a branch the wrong way, or having it be many commits instead of 1, ... etc. I can change any of this and do another PR if it's wrong.
I can spend 1 more week on anything this is missing, but after that I have to do other things.

I am closing this and will put the checkbox feature in a separate PR so there will be 2. The first PR will simply make the mod load in 2.0. Also I will rewrite the checkbox feature to change fewer functions by not passing "surface" around so much.


build: upgrade spring version 2.6.10

Summary

Details

Context

Checklist:

[ ] I've added tests for my code.
[ ] Documentation has been updated accordingly to this PR

Related issue :

Thank you @jitendracsc for the proposal


Feature Request: Saved Discord Webhooks

When setting up schedules with Discord messaging tasks, it would be helpful if we could reference saved webhooks that we can easily reuse in multiple tasks without having to re-enter the webhook each and every time.
Would be best if they're account specific, but could be shared with other users. That way, they could use your webhook within AMP, but not have access to it.

That's a great idea.
In my case, posting most Micecraft sepeciffic massagest to Discord does not work.
I can't find any documentation


fix: remove regex, parse remote as URL

[ ] Added an entry to CHANGELOG.md if this change could be valuable to users

Codecov Report

Merging #2075 (b78ee04) into master (0ae86e6) will increase coverage by 0.05%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories.  Learn more


Fix Warnings for >=4.24

Since the 4.24 update there were some warnings regarding the library paths. With using the UE_4_X_OR_LATER this can be suppressed / fixed while remaining functional for lower versions.
I haven't checked compiling for versions below 4.24 though.

Thank you for the contribution. I'm going to check how well this compiles against 4.23->4.25  when I have completed my current task(s)

So I have finally come around to do this. I tried compiling in 4.24 and 4.23, but the 4.23 didn't link anymore after your changes.
It's a simple fix:
#if UE_4_24_OR_LATER
			PublicAdditionalLibraries.Add(Path.Combine(LibrariesPath, "modio.lib"));
#else
			PublicLibraryPaths.Add(LibrariesPath);
			PublicAdditionalLibraries.Add("modio");
#endif

should be
#if UE_4_24_OR_LATER
			PublicAdditionalLibraries.Add(Path.Combine(LibrariesPath, "modio.lib"));
#else
			PublicLibraryPaths.Add(LibrariesPath);
			PublicAdditionalLibraries.Add("modio**.lib**");
#endif

I can easily fix that on my side, but if you want to fix the pull request, I would be happy to merge it into the repo so that credit goes to the right source

Just ensure, does the thumbs up mean that you want me to fix it, if so react with 👍

This has been resolved when I did some other work on cross platform code


Integrate all core repositories into devs slack channel

Story
As a core dev I want to have all the relevant repositories integrated with Slack so I can keep track of project issues and PRs more easily.

Implementation details
NB Requires organization level Github access.
Add the repositories to slack using /github org/repo command in-app.
Priority
3/5
We want this, but it is not urgent.
Scope
No changes to codebase
Size
10 minutes once access to GSF account is provided
What does "done" look like?
Repos are integrated in channel
Does this require updates to documentation or other materials??
No
What testing is required?
None
Is this a known/expected update?
Discussed at retro 1-12-23

Closing as completed


feat(tier4_localization_launch): declare param path argument

Description
Set tier4_localization_launch_param_path argument

Related links

localization: https://github.com/autowarefoundation/autoware.universe/pull/1404
planning: https://github.com/autowarefoundation/autoware.universe/pull/1337
system: https://github.com/autowarefoundation/autoware.universe/pull/1411
vehicle: https://github.com/autowarefoundation/autoware.universe/pull/1416

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kminoda
Please wait for @yukkysaito -kantoku's brief review just in case


Convert to using bot emojis rather than relying on server emojis

waiting for 2.5 release which should bring support
https://github.com/Rapptz/discord.py


Consider using different namespaces for provider internal code

And making this guidance for provider writers so that it is much easier to tell if a provider is accidentally using EF internal code. See https://github.com/PomeloFoundation/Pomelo.EntityFrameworkCore.MySql/issues/497#issuecomment-369929292
/cc @roji @ErikEJ @caleblloyd

Excellent proposal, I use the same convention

Agree, I do use the same namespaces as EF Core and it's easy to see how a reference to an internal namespace could slip into there.

I made this change, and can now see that we have a lot of references to upstream internal classes.  How are you guys dealing with this @ErikEJ @roji?  Copying these files into your provider?
https://github.com/PomeloFoundation/Pomelo.EntityFrameworkCore.MySql/issues/497#issuecomment-372128480

I haven't made the change, but I wouldn't be surprised to find out I have some references to EF Core internal classes. I work hard to keep the provider in sync with EF Core relatively quickly, so I'm sure that has prevented breakage as I adapt to changes in internal EF Core code...

Coming to think of it, aside from very specific extension classes that should automatically light up for users, why shouldn't all provider classes be in their own namespace? What's the gain in having even non-public classes in Microsoft.EntityFrameworkCore?

I have the following problematic uses of .Internal namespaces, that I cannot see how to avoid, but maybe I am doing something wrong? @ajcvickers
https://github.com/ErikEJ/EntityFramework.SqlServerCompact/blob/master/src/Provider40/Extensions/SqlCeServiceCollectionExtensions.cs
https://github.com/ErikEJ/EntityFramework.SqlServerCompact/tree/master/src/Provider40/MetaData/Conventions/Internal
https://github.com/ErikEJ/EntityFramework.SqlServerCompact/blob/master/src/Provider40/MetaData/Internal/SqlCeInternalMetaDataBuilderExtensions.cs
https://github.com/ErikEJ/EntityFramework.SqlServerCompact/blob/master/src/Provider40/Query/Internal/SqlCeQueryCompilationContext.cs
https://github.com/ErikEJ/EntityFramework.SqlServerCompact/blob/master/src/Provider40/Query/Internal/SqlCeQueryCompilationContextFactory.cs

I looked at MySQL solution to figure out what kind of references to EFCore Internal classes are.

Conventions are Internal, when provider has to add more conventions or extend one of the core ones, they are required to use .Internal
ProviderMetadataAnnotations/BuilderExtensions classes are using InternalMetadataBuilders/ConfigurationSource which requires .Internal
IRelationalCommandBuilder implementes an generic interface with IndentedStringBuilder which is .Internal
Implementing IRelationalCommandBuilder requires implementation of IRelationalParameterBuilder too since RelationalParameterBuilder is .Internal.
CoreStrings/RelationalStrings references from exception throwing.
Logging using EF core methods from provider code.
TableSelectionSet class in RevEng. perhaps providers are free to choose how they want to do it. (We removed this class in 2.1)
Utilities like DisplayName/ShortDisplayName

Some of above are actually used in our providers too.
Some of them are easy to fix in a way that provider may need to copy the code (like utilities/TableSelectionSet).
CommandBuilding can be partially fixed using implementing all related interfaces.
Conventions/Annotations framework has no other way and it is like that in SqlServer provider too.

From SqlCe, apart from 1-2 from last post about Convention/Annotations,
QueryCompilationContext requires ILinqOperatorProvider. That interface & implementation both are .Internal. So overrideing QueryCompilationFactory forces use of .Internal.

@smitpatel  But must I override QueryCompilationContext at all??

@ErikEJ - Yes. The interfaces are public so providers can override it. Perhaps one of the parameter type left out from public by error.

@caleblloyd @ErikEJ @smitpatel @roji This is great data! We will look through these and consider what to do about each case. The goal here is not to force providers to copy "a lot" of code to avoid using internals, but instead to identify and fix holes in the public surface that send providers down the internal code path. However, that being said, there likely is some places that a small amount of utility code will need to be copied. If it turns out to be too tempting to use this code anyway, then we might make it really internal like we have for a bunch of the Reflection helpers--so that we are free to add to and remove these as needed.
For conventions, we know that these are internal but must be used by providers in this way until we do the work to make them public. We take the responsibility to not break these APIs (without coordination with providers) even though they are technically internal.

@roji I have no problem with changing the namespaces of the public provider code if you want to do so, although it will likely technically result in a breaking change at this point, since these are public types that have been shipped. I honestly can't quite remember why we choose this namespace approach for our providers. I think it made some things look "cleaner" in some way, but I don't think it was for any specific functional reason.

OK.
At least for my side (with the somewhat-reduced backwards compatibility constraints :)), I think I'll be changing the namespace for the entire provider, aside from extension classes which are obviously meant to be publicly-accessible (e.g.UseNpgsql()). I'll report back here on any Internal dependencies on EF Core that I find.

FYI: I have file copied Check.cs..

It did not mention Strings (last paragraph above), but I have the same issue, just did not mention it.

@ErikEJ @roji @caleblloyd I filed #11266 to capture the work we plan to do in 2.1 with regard to provider use of internal types. Can we continue any discussion there?

We discussed in design meeting and arrived at follow conclusions:
For our providers, we will use the package name as root namespace. So for SqlServer, it becomes Microsoft.EntityFrameworkCore.SqlServer.
The sub namespaces will append after the root following same hierarchy as EFCore.
The changes we have made as following

Provider
Root Namespace

SqlServer
Microsoft.EntityFrameworkCore.SqlServer

Sqlite
Microsoft.EntityFrameworkCore.Sqlite

InMemory
Microsoft.EntityFrameworkCore.InMemory

OracleProvider
Microsoft.EntityFrameworkCore.Oracle


JSON converters are converting to lowercase.

The GalleryItem and NotificationConverters are converting the json string to lowercase.
Imgur Ids are case sensitive.

Fixed in v3.2.0-alpha1


Some characters are sticking together after PDF print

Example of the issue:

Code snippet:
try {
  const browser = await launchChromium({ headless: true })
  const page = await browser.newPage({
    viewport: {
      width: 868,
      height: 500,
    },
  })
  const nextURL = new URL(url)
  await page.goto(nextURL.toString())

  const { getHeaderTemplate, getFooterTemplate, margin } = templates[type]
  const [headerTemplate, footerTemplate] = await Promise.all([
    getHeaderTemplate(templateProps),
    getFooterTemplate(templateProps),
  ])

  const buffer = await page.pdf({
    displayHeaderFooter: true,
    headerTemplate,
    footerTemplate,
    margin,
    printBackground: true,
  })

  const compressed = zlib.gzipSync(buffer)
  const contentEncoding = 'gzip'

  const filename = 'filename.pdf'
  const s3Key = 'mykey/'.concat(filename)
  const contentType = 'application/pdf'
  const putObjectParams = {
    Bucket: s3Bucket,
    Key: `public/${s3Key}`,
    ContentType: contentType,
    ContentEncoding: contentEncoding,
    Body: compressed,
  }

  await Promise.all([s3.putObject(putObjectParams).promise(), browser.close()])

  return {
    statusCode: 200,
    headers: defaultHeaders,
    body: {
      s3Key,
      filename,
      contentType,
      contentEncoding,
    },
  }
} catch (error) {
  console.error(error)
  return {
    statusCode: 500,
    headers: defaultHeaders,
    body: { error: error.message },
  }
}

Further explanation:

A URL is sent to the lambda.
The lambda spins up a headless Chromium browser and navigates to the URL.
The page is captured as a pdf and sent to S3.
The lambda returns the S3 key, which allows the user to download the PDF from S3.

Help & Guidance please.

Hey @sh4nnongoh , did you have any luck solving this problem? I've been facing the same issue and can't seem to get any closer to solving it.

@sh4nnongoh I've been able to work around the issue with this, hope it helps!
browser = await playwright.launchChromium({
            args: [
                ...playwright.getChromiumArgs(true),
                '--font-render-hinting=none'
            ]
        });

Seems to be a common chromium issue, found the solution on puppeteer https://github.com/puppeteer/puppeteer/issues/2410


Added support to exclude the properties in the search

Added support to exclude the properties while searching - #Fix-40

Since we now support filtering in nested objects, we need to be able to "exclude" the nested properties.
I think we can do this by passing a filtering the 'exclusionList' to the checkInside function and check there with ===.

@aVolpe Have added one more condition to the If block:
excludes.includes(property)

@prashantpimpale93 your test aren't passing in the latest version.
Failures:
1) Pipe: Default Ignore the property in the search
  Message:
    Expected $.length = 2 to equal 1.
    Expected $[1] = Object({ a: Object({ b: 'e', c: Object({ b: 'd' }) }) }) to equal undefined.
  Stack:
    Error: Expected $.length = 2 to equal 1.
    Expected $[1] = Object({ a: Object({ b: 'e', c: Object({ b: 'd' }) }) }) to equal undefined.
        at UserContext.<anonymous> (/Users/arturovolpe/develop/avolpe/ng2-search-filter/src/ng2-filter.pipe.spec.ts:38:50)

5 specs, 1 failure
Finished in 0.021 seconds

Changing to this in the pipe seems to fix it:
           if (checkInside(item[property], term)) {
             return true;
           }
-        }
-        if (item[property].toString().toLowerCase().includes(toCompare)) {
+        } else if (item[property].toString().toLowerCase().includes(toCompare)) {
           return true;
         }

@aVolpe okay will test it and commit again!

@aVolpe Updated the code and committed!
Have a look!

@aVolpe Did you review the code changes?

I think the changes are a first aproach to the issue, maybe in a future we can add more complex filters like:
"user.name", // to only filter  the name of the user
"user.ci*"

@aVolpe Thanks a lot!


The function in the example has an error

Hi,
There is an error in Usage about ReverseString   (step 2).
It should be Array.Reverse(charArray); instead of Array.Reverse(input.ToCharArray());.
Please have a try.
Thanks
Remember, the details make the difference

That's coooool


"CrashHandlerException: Program crashed" when quitting the godot app

Godot version
4.0.1-stable
System information
Windows 10, Vulkan API 1.3.205 - Forward+ - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1080 Ti
Issue description
Use gdextension demo official version godot-4.0.1-stable. open godot engine 4.0.1 and the project on the command line, run and then close, and "CrashHandlerException: Program crashed" will appear on the command line. If not, please try a few more times, it seems to have a small chance of ending normally

The gdextension c++ project is very simple, just call the ResourceLoader::get_singleton()->load("res://test_loader.tscn");code in c++.
Steps to reproduce
Open godot engine 4.0.1 and the project on the command line, run and then close.
Minimal reproduction project
test_gdextension_resource_loader.zip

Duplicate of https://github.com/godotengine/godot-cpp/issues/889


Support supplementaryViews created from storyboard

Miss this in the previous PR #92. Will add unit test soon.

Coverage decreased (-0.6%) to 85.454% when pulling c22d81c817884070bd56b6f2e886382a29db77c2 on zhubofei:supplementary-storyboard into 920aad6a698034a0ca0a4a7a3e3f6fbce9522a31 on Instagram:master.

@zhubofei nice catch! Before we land this, can we add some unit tests to make sure they are working? Looks like you added views to the test storyboard, just need like one test and we're g2g.

@rnystrom I'm working on that right now and should finish by the end of the day. I do notice one thing, though. In the source code, all the *ItemControllers have been renamed to *SectionController. But in unit tests, they stay with Item. Do you keep their name on purpose?

@zhubofei ha, leftover from naming refactor. Surprised those never showed up! Yes we should move them in a separate PR. #126

A simple test with a lot of pain. Finally. 😂

Coverage decreased (-0.07%) to 86.013% when pulling 71e003c439f18e14ee625238561ec1aa00041444 on zhubofei:supplementary-storyboard into 920aad6a698034a0ca0a4a7a3e3f6fbce9522a31 on Instagram:master.

@rnystrom Should I test that annoying 0.07% in StackSectionController? 😂

@zhubofei 😂  no use, don't worry about it.
Small nit then we're g2g on this!

Coverage decreased (-0.07%) to 86.013% when pulling ec9cc5e1a75e94a6c78d9ed1aebe3d3eab013eb8 on zhubofei:supplementary-storyboard into 920aad6a698034a0ca0a4a7a3e3f6fbce9522a31 on Instagram:master.

@zhubofei it looks like IGListKit.xcodeproj/project.pbxproj got out of sync. Mind rebasing and fixing that up?

@rnystrom Just did. I think we're g2g.

Coverage decreased (-0.2%) to 86.297% when pulling 3467a1ab7109671808c657cc753670eebcd8630d on zhubofei:supplementary-storyboard into 9e8ad9626ab91326b6bbb75cb6927f8dbe134896 on Instagram:master.


Greenkeeper/initial

I'll follow up with the lockfile in a separate PR: https://github.com/Shopify/quilt/issues/162

@TheMallen with the new linting config, vscode and the yarn lint command have some conflicts. If you have any ideas as to why that might be, please let me know.

I figured it out; once the packages for eslint have been upgrades, I just had to restart vs code in order for the changes to apply properly.

[x] tests passing
[x] linting passing
[x] build passing

:shipit:


Feat/header footer code

Header and footer code injection
Closes #134

Should I increase the Width for the code injection section?


Git push failing on logsec Ubuntu

git plugin fails to push. It gives this error: "push error: fatal: cannot exec 'ssh': Permission denied fatal: unable to fork"
gh installed, cloned repo using gh command
pushing with no issues in terminal

I'm getting the same error. Any ideas what might be causing it?

Same here.
Also haven't recognized the global git user.email config and had to set locally.
Recreated ssh keys, and added to the ssh agent, but didn't help.


Install rancher app to specific namespace

What kind of request is this (question/bug/enhancement/feature request):
question
Steps to reproduce (least amount of steps as possible):
I want to install rancher app to specific namespace So that gitlab private registry credentials could be used. I'm using rancher cli command.
To install the app I'm using:
rancher app install --answers ans.yaml rancher-app

Result:
It installs the app on its own namespace and it errors cause the images don't get pulled from the gitlab registry.
I searched for away to make the creds secret to be available for all namespace but I don't think this could be done.
I can't find anything on the docs  regards this issue.

@superseb  Is this possible  in the current version of rancher?

Please use --namespace https://github.com/rancher/cli/blob/master/cmd/app.go#L168


docs(api-docs): update streamId documentation

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Change python.version from python3 to default

This addon specifies python3 as the python version in a few stanzas causing validation errors on a few conflicting files.
For this app to be dual compatible with <8.0 I would recommend changing the python.version value in the referenced stanzas to the value of "default" to use the system version of python rather than forcing python3
This is what I get with after restarting a fresh install of Splunk Enterprise <IP_ADDRESS> on Mac with Webex Meetings TA 1.1.1
$ ./splunk restart
Stopping splunkd...
Shutting down.  Please wait, as this may take a few minutes.
..
Stopping splunk helpers...

Done.

Splunk> Winning the War on Error

Checking prerequisites...
	Checking http port [8000]: open
	Checking mgmt port [8089]: open
	Checking appserver port [<IP_ADDRESS>:8065]: open
	Checking kvstore port [8191]: open
	Checking configuration...  Done.
	Checking critical directories...	Done
	Checking indexes...
		Validated: _audit _internal _introspection _telemetry _thefishbucket history main summary
	Done
	Checking filesystem compatibility...  Done
	Checking conf files for problems...
		Invalid key in stanza [general_service] in /Applications/Splunk/etc/apps/ta-cisco-webex-meetings-add-on-for-splunk/default/inputs.conf, line 3: python.version  (value:  python3).
		Invalid key in stanza [history_service] in /Applications/Splunk/etc/apps/ta-cisco-webex-meetings-add-on-for-splunk/default/inputs.conf, line 11: python.version  (value:  python3).
		Invalid key in stanza [admin_external:ta_cisco_webex_meetings_add_on_for_splunk_settings] in /Applications/Splunk/etc/apps/ta-cisco-webex-meetings-add-on-for-splunk/default/restmap.conf, line 10: python.version  (value:  python3).
		Invalid key in stanza [admin_external:ta_cisco_webex_meetings_add_on_for_splunk_general_service] in /Applications/Splunk/etc/apps/ta-cisco-webex-meetings-add-on-for-splunk/default/restmap.conf, line 16: python.version  (value:  python3).
		Invalid key in stanza [admin_external:ta_cisco_webex_meetings_add_on_for_splunk_history_service] in /Applications/Splunk/etc/apps/ta-cisco-webex-meetings-add-on-for-splunk/default/restmap.conf, line 22: python.version  (value:  python3).
		Your indexes and inputs configurations are not internally consistent. For more information, run 'splunk btool check --debug'
	Done
	Checking default conf files for edits...
	Validating installed files against hashes from '/Applications/Splunk/splunk-<IP_ADDRESS>-40b15aa1f501-darwin-64-manifest'
	All installed files intact.
	Done
All preliminary checks passed.

Starting splunk server daemon (splunkd)...
Done

@byrondowns Thanks for reaching out.
Newer AppInspect requirements mandate python3 in the conf file.

appinspectreferencetopics/splunkappinspectcheck

python.version=python3 stanza will not force python3 and shall fallback to python2 for Splunk version 7.2 < python2 < 8.0
You may ignore those errors.  Let us know if this answers your question/

@byrondowns  when you have a moment, let us know if this answers your question.

@byrondowns  marking this as closed because of inactivity.
Should you have further questions, do not hesitate to reach out!


Added tagging support and toolbars on list views.

Added checkboxes & toolbar buttons Orchestration Templates list views, so add/copy/remove features can be performed from list views. Made changes to methods to handle buttons coming in from list views.
Added tagging support for Orchestration Templates. Adjusted existing code for Catalog Items tagging to be reused for Orchestration tagging support.
Renamed toolbars for Orchestration Templates to be consistent with other toolbar names.
Changed find_checked_items to do .to_s on the params as params coming in from spec tests send up params as symbols which was causing tests to blow up, added spec tests to test the change.
Added spec tests to support changes in this PR.

Issue #2038
@dclarizio please review/test.

@gmcculloug Can you or someone from your group also give this a try?  Thx, Dan

@bzwei Please review

Cloudformation Template list view updated on column is empty. It is populated in Heat Template or all templates list view.
Otherwise the new feature looks good.

@bzwei please rebase and try again, fixed a typo in yaml

verify problem solved!


Large memory growth in long running processes

Heyo!
First off, great project! We are parsing postgres queries in Go and came across this via sqlc due to not wanting to add a CGO dependency.
This for the most part works great, however, it seems to have a large amount of memory growth when running in a long running process.
Below is a heap snapshot from calling it four times.

Not doing anything crazy here, just passing in a sql statement so that we can qualify where conditions with their respective schema and table.
Maybe we're doing something wrong, but I am seeing massive memory growth correlated to the amount of calls we make to this library.
What makes it even more difficult is that it doesn't seem to free the memory for 10 minutes.
Here's an snapshot of some memory consumption I clocked when running it.

You can see how we're calling it here:
https://github.com/nucleuscloud/neosync/blob/main/worker/pkg/query-builder2/querybuilder.go#L263

Hi @nickzelei - thanks for the report and sorry for the issut. While performance isn't a focus for this library (yet) this still looks worth checking. Actually our build is probably showing the same behavior in the benchmark and I've been putting it off since the common usage with the sqlc binary isn't long living. But time to stop putting it off 😅

Hi @nickzelei - I have pushed a change that seems to have resolved the memory issues we were seeing in the benchmark workflow here, so I am hopeful it helps with your issue too (basically, sync.Map doesn't interact well when wazero uses mmap for memory allocation as it is off-GC). Would you be able to try it and see if it helps?

Hi @anuraaga appreciate the quick response and apologies I haven't had much time to look into this at all. But I did want to atleast ack your message and let you know that I appreciate your quick response. I'll check into it when I am able to.


Release 1.1.4

New: Bolts for iOS is easily importable from Swift code (via import Bolts).
New: Added BFTask +taskForCompletionOfAllTaskResults.
New: Added faulted property on BFTask.
New: Made BFTaskErrorDomain and BFTaskMultipleExceptionsException constants publicly available.
New: BFTask -description now shows completed/cancelled/faulted status of a task.

cc @grantland

LGTM


Highlighter minus rejected components

Same as highlight-clean except the sections that were intentionally deleted (collapsible list in topic picker) have been removed.

I think the radium dependency is missing from package.json:
npm install radium --save-dev

After I install radium and then run npm run dev, I get:
ERROR in ./~/radium/lib/append-important-to-each-value.js
Module build failed: Error: Couldn't find preset "es2015-loose" relative to directory "/home/norman/goodly/TextThresher/node_modules/radium"

There is a dependency on babel-preset-es2015-loose I believe.
On Mon, Aug 29, 2016 at 2:19 PM, Norman Gilmore<EMAIL_ADDRESS>wrote:

After I install radium and then run npm run dev, I get:
ERROR in ./~/radium/lib/append-important-to-each-value.js
Module build failed: Error: Couldn't find preset "es2015-loose" relative
to directory "/home/norman/goodly/TextThresher/node_modules/radium"
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Goodly/TextThresher/pull/8#issuecomment-243259789,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AG7w6aN8kz4rMOfYz7-GaTn8lKUpfTnLks5qk0zGgaJpZM4JuSyV
.

@stevenelleman and I were cleaning up what we thought were unused branches on Friday from within the git repo, so I didn't realize this branch had a lot of active discussion on it.
I'm restoring this in case I made a mistake and this is still work-in-progress.

I believe this is still a WIP though I haven't seen any progress in some time. @stevenelleman what's the status on this refactor?

@stevenelleman is working on highlighter in #15. Keeping this open since it has relevant comments.

no reason to keep this open as it is duplicated in https://github.com/Goodly/TextThresher/pull/15


.overwatch command failing to find users

The .overwatch command as far as I know requires no special API, and I don't get any error messages explaining that I don't have the necessary API like I do with other commands. Instead, it just simply tells me that it can't find my account in Overwatch. This happens all the time, with everyone who tries using the command.

It's not that we're just inputting something incorrectly, we've made sure that all information is correct.

Yes, ow api will be back tomorrow hopefully.


vcpkg install x86 system.

vcpkg install libheif. then

why install x86-windows? my computer is x64.
please let me know for newbie.

See #53

Thank for your help, i solving this problem!!

See #53

awesome! how can you find it?

See #53
awesome! how can you find it?


[WAF - Reliability Review] - avm/res/app-configuration/configuration-store

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Module Name
avm/res/app-configuration/configuration-store
(Optional) Module Name if not listed above
No response
(Optional) Module Version
No response
Description
Hello team,
The AVM core team are performing a review of all AVM modules to ensure they are aligned to the WAF reliability pillar by default, and in the future they will be checking against other WAF pillars also. This is a requirement we have before we can go v1.0.0 for any AVM module.
Links to AVM specs:

https://azure.github.io/Azure-Verified-Modules/specs/shared/#id-sfr2---category-composition---waf-aligned
https://azure.github.io/Azure-Verified-Modules/specs/shared/#id-sfr5---category-composition---availability-zones

Links to WAF Reliability, APRL & Product Docs recommendations used for analysis:

https://learn.microsoft.com/azure/well-architected/reliability/checklist
https://learn.microsoft.com/en-us/azure/azure-app-configuration/faq?toc=%2Fazure%2Freliability%2Ftoc.json&bc=%2Fazure%2Freliability%2Fbreadcrumb%2Ftoc.json#how-does-app-configuration-ensure-high-data-availability

Whilst reviewing this module the following issues have been found that are non-compliant with the AVM specs that need to be resolved in a new version release that can be seen in the PSRule for Azure run here: https://github.com/Azure/bicep-registry-modules/actions/workflows/avm.res.app-configuration.configuration-store.yml#summary-23300173139

The geo-replica PSRule check may need a suppression rule creating as it may only be suitable for a pattern module and not a resource module. The AVM .ps-rule directory can be found here:  https://github.com/Azure/bicep-registry-modules/tree/main/avm/utilities/pipelines/staticValidation/psrule/.ps-rule

Please make these changes in a PR to the parameter defaults so a new minor version can be published.
Thanks
The AVM Core Team
(Optional) Correlation Id
No response

Note, this module is currently orphaned: https://github.com/Azure/Azure-Verified-Modules/issues/731

Seems "Microsoft.AppConfiguration/configurationStores/replicas" needs to be added.

@JFolberth is actively working on fixing this issue through PR #1696


Specify host port when different from scheme default port

When port is not the same as the standard port for the scheme, it must be specified in the HTTP Host header  (https://tools.ietf.org/html/rfc2616#section-14.23)

@tanner0101 Everything is now fixed 🙂

Hey @adtrevor, you just merged a pull request, have a coin!
You now have 3 coins.


Fix for bitflags!

refs https://github.com/rust-lang/rust/pull/21254

Has removing bitflags landed on master?

Yeah, it's in the latest nightly.

Merged, thanks for the PR <3

^_^ thx


I encountered a problem with file overwriting when using LTO=yes

I encountered a problem with file overwriting when using LTO=yes
Nuitka version: 2.2.3
Linux version: Debian12.5
Python version: 3.11
Build command: python3 -m nuitka --follow-imports --standalone --show-scons --lto=yes --static-libpython=yes --include-package=uvicorn,gunicorn,pydantic,logging --output-dir =build/app.dist app.py

(.venv) root@erp:~/Zero_ERP# python -m nuitka --version
2.2.3
Commercial: None
Python: 3.11.2 (main, Mar 13 2023, 12:18:29) [GCC 12.2.0]
Flavor: Debian Python
Executable: /root/Zero_ERP/.venv/bin/python
OS: Linux
Arch: x86_64
Distribution: Debian 12
Version C compiler: /usr/bin/gcc (gcc 12).

How did you install Nuitka and Python
I used the Python environment that comes with Debian and installed the virtual environment created by Python-Venv using Apt.
Question code:
DEFAULT_CONFIG = {....}
#The above is the code that defines the default configuration
# Problem code load_config
def load_config():
     try:
         with open('config.yaml', 'r') as file:
             config = yaml.safe_load(file)
     exceptException:
         key = jwk.JWK.generate(kty='RSA', size=4096)
         DEFAULT_CONFIG['PRIVATE_KEY_JSON'] = key.export_private()
         DEFAULT_CONFIG['PUBLIC_KEY_JSON'] = key.export_public()
         config=DEFAULT_CONFIG
         with open('config.yaml', 'w') as file:
             yaml.dump(DEFAULT_CONFIG, file)
     return config
#The following is the code to read the configuration and save it as a constant
CONFIG = load_config()
HOST = CONFIG['HOST']

Details of the problem encountered:
I use FastApi to build the backend of my web application. When I write my program, I need to read the relevant configuration information from config.yaml, so I use open to open the Yaml file, but when I enable LTO =Yes, this configuration file will be read and written repeatedly. For example, I have modified my configuration file now. When the LTO-optimized program is running, it will not read the contents of the file, but will overwrite and write the configuration file. As a result, all configurations of the program will not be used. When I turn off LTO optimization, the problem is solved. Another strange phenomenon is that when I use logging or print methods to output Chinese characters, using LTO optimization will convert the Chinese characters into Unicode encoding for output.
The above are the problems I encountered when using Nuitka. Nuitka is a good project, and I would like to thank all developers for their efforts. I hope this project gets better and better

我想我应该去捕获这个异常，请稍等我去查看这个异常输出信息

I think I should catch this exception. Please wait a moment while I check the output information of this exception.

The Unicode code is Chinese, and the translated meaning is "Enterprise WeChat configuration error, please check the relevant configuration of Enterprise WeChat." Because the relevant configurations of Enterprise WeChat are stored in config.yaml, the error reported ultimately is the code for loading config above. 'ascii' codec can't decode byte 0xe6 in position 146: ordinal not in range(128)
root@erp:~/test# ./app.bin
'ascii' codec can't decode byte 0xe6 in position 146: ordinal not in range(128)
\u4f01\u4e1a\u5fae\u4fe1\u914d\u7f6e\u9519\u8bef\uff0c\u8bf7\u68c0\u67e5\u4f01\u4e1a\u5fae\u4fe1\u76f8\u5173\u914d\u7f6e
Traceback (most recent call last):
  File "/root/test/app.py", line 18, in <module>
ImportError: cannot import name 'wechat_push' from 'ext.ext' (/root/test/ext/ext.py)

error code:
'ascii' codec can't decode byte 0xe6 in position 146: ordinal not in range(128)

In my testing, this is a possible fix: specify UTF-8 encoding, but I still don't understand why this problem will not occur without turning on LTO optimization.
def load_config():
    try:
        with open('config.yaml', 'r', encoding='utf-8') as file:
            config = yaml.safe_load(file)
    except Exception as e:
        # Generate a new key pair
        print(e)
        key = jwk.JWK.generate(kty='RSA', size=4096)

        # Export the private key and public key
        DEFAULT_CONFIG['PRIVATE_KEY_JSON'] = key.export_private()
        DEFAULT_CONFIG['PUBLIC_KEY_JSON'] = key.export_public()

        config = DEFAULT_CONFIG
        with open('config.yaml', 'w', encoding='utf-8') as file:
            yaml.dump(DEFAULT_CONFIG, file)

    return config

The default encoding is sys.getfilesystemencoding(). Maybe print it in compiled variants of your system and see if there is a difference. I doubt there would be a difference.

The default encoding is sys.getfilesystemencoding(). Maybe print it in compiled variants of your system and see if there is a difference. I doubt there would be a difference.

Thank you very much for your suggestion, I added sys.getfilesystemencoding() to test run. The following is the modified code.
As you can see from the program output below, using LTO optimization, the default will become ascii instead of UTF-8.
This may be the reason why Chinese characters become Unicode, and it may also be the cause of the error 'ascii' codec can't decode byte 0xe6 in position 146: ordinal not in range(128) reason
In addition, I will attach the log of Github Action. This example is compiled with Github Action. After my testing, whether using local Debian12 using nuitka installed with pip or using Github Action directly, this problem always exists.
job-logs.txt
def load_config():
    try:
        systemencoding = sys.getfilesystemencoding()
        print(f"Load_Config_Nomal:{systemencoding}")
        with open('config.yaml', 'r') as file:
            config = yaml.safe_load(file)
    except Exception as e:
        # Generate a new key pair
        systemencoding = sys.getfilesystemencoding()
        print(f"Load_Config_Exception:{systemencoding}")
        print(e)
        key = jwk.JWK.generate(kty='RSA', size=4096)

        # Export the private key and public key
        DEFAULT_CONFIG['PRIVATE_KEY_JSON'] = key.export_private()
        DEFAULT_CONFIG['PUBLIC_KEY_JSON'] = key.export_public()

        config = DEFAULT_CONFIG
        with open('config.yaml', 'w') as file:
            yaml.dump(DEFAULT_CONFIG, file)

    return config

lto=yes
root@erp:~/ltoyes-18# ./app.bin
Load_Config_Nomal:ascii
Load_Config_Exception:ascii
'ascii' codec can't decode byte 0xe6 in position 146: ordinal not in range(128)
\u4f01\u4e1a\u5fae\u4fe1\u914d\u7f6e\u9519\u8bef\uff0c\u8bf7\u68c0\u67e5\u4f01\u4e1a\u5fae\u4fe1\u76f8\u5173\u914d\u7f6e
Traceback (most recent call last):
  File "/root/ltoyes-18/app.py", line 18, in <module>
ImportError: cannot import name 'wechat_push' from 'ext.ext' (/root/ltoyes-18/ext/ext.py)

Use Cpython directly and lto=no
Load_Config_Nomal:utf-8
[2024-05-22 08:50:33 +0800] [2708] [INFO] Starting gunicorn 22.0.0
[2024-05-22 08:50:33 +0800] [2708] [INFO] Listening at: http://<IP_ADDRESS>:8000 (2708)
[2024-05-22 08:50:33 +0800] [2708] [INFO] Using worker: uvicorn.workers.UvicornWorker
[2024-05-22 08:50:33 +0800] [2725] [INFO] Booting worker with pid: 2725
[2024-05-22 08:50:33 +0800] [2725] [INFO] Started server process [2725]
[2024-05-22 08:50:33 +0800] [2725] [INFO] Waiting for application startup.
[2024-05-22 08:50:33 +0800] [2725] [INFO] Application startup complete.
[2024-05-22 08:50:33 +0800] [2727] [INFO] Booting worker with pid: 2727
[2024-05-22 08:50:33 +0800] [2727] [INFO] Started server process [2727]

Load_Config_Nomal:utf-8
Load_Config_Exception:utf-8
[Errno 2] No such file or directory: 'config.yaml'
企业微信配置错误，请检查企业微信相关配置
Traceback (most recent call last):
  File "/root/Zero_ERP/app.py", line 18, in <module>
    from ext.ext import wechat_push
ImportError: cannot import name 'wechat_push' from 'ext.ext' (/root/Zero_ERP/ext/ext.py)

Github Action
jobs:
  build:
    strategy:
      matrix:
        os: [ ubuntu-20.04 ]

    runs-on: ${{ matrix.os }}

    steps:
      - name: Check-out repository
        uses: actions/checkout@v4

      - name: Setup Python
        uses: actions/setup-python@v4
        with:
          python-version: '3.11'
          architecture: 'x64'
          cache: 'pip'
          cache-dependency-path: |
            **/requirements*.txt

      - name: Install Dependencies
        run: |
          python -m pip install --upgrade pip
          pip install -r requirements.txt

      - name: Build Executable
        uses: Nuitka/Nuitka-Action@main
        with:
          nuitka-version: main
          script-name: app.py
          company-name: KooKoo.top
          product-name: ZERO_ERP
          file-version: ${{ github.ref_name }}
          product-version: ${{ github.ref_name }}
          standalone: true
          show-scons: true
          onefile: false
          static-libpython: yes
          lto: yes
          include-package: |
                    uvicorn
                    gunicorn
                    pydantic
                    logging

This must be GitHub Python being strange. I think it might be unable to load extension modules when you use static libpython, and some codeds, specifically asian ones, might be affected. I see them being exxtension modules normally. This may or may not happen with LTO. You best mess with the static-libpython setting if you know what you are doing.
LTO will inline things from there and be able to drop unused things, but can be wrong about that.
What I think is kind of missing from Nuitka is a sort of auto-detection by defined experiments that allows to decide if the static libpython is good or not. And if LTO works or not. This is hard to make though.

This must be GitHub Python being strange. I think it might be unable to load extension modules when you use static libpython, and some codeds, specifically asian ones, might be affected. I see them being exxtension modules normally. This may or may not happen with LTO. You best mess with the static-libpython setting if you know what you are doing.
LTO will inline things from there and be able to drop unused things, but can be wrong about that.
What I think is kind of missing from Nuitka is a sort of auto-detection by defined experiments that allows to decide if the static libpython is good or not. And if LTO works or not. This is hard to make though.

Thank you for your reply. However, I don't think the issue is related to the static-libpython setting, because this encoding problem still occurs even when I don't specify the static-libpython setting (i.e., when it is set to NO). I believe the ultimate solution to this problem is to add encoding='utf-8' to with open, which prevents my app from throwing errors. If possible, could you add this point to the "Solutions to the Common Issues" section on the official website? It might help others as well. Thank you very much for your help。

Nah, it's a broken configuration of Python if that's needed. I don't know how to deal with people who enable modes that don't work yet.


How to convert .tex to .pdf ?

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Hello @aasthavar !
Make sure to follow the instructions for setting up the project on your computer: https://github.com/thehale/expressive-resume#installation--setup
After opening the project inside of the "Devcontainer", saved edits to your .tex file will automatically trigger the build of a .pdf of the result.
If you have further difficulties, feel free to re-open the issue with additional details (e.g. error messages, screenshots, etc.)


Backend search capacity in range instead of >=

As per the front end tool-tip suggested, we should only return tank capacity in water heater and capacity in dryer within a certain range instead of all the appliances that has bigger capacities.

Let me know your thoughts on this.

Let me know your thoughts on this.

For capacity, I believe our next step is to let users choose an upper and a lower value as @datu925 suggested.

Let me know your thoughts on this.

For capacity, I believe our next step is to let users choose an upper and a lower value as @datu925 suggested.

The change in this PR adds such change in the backend. I changed the front-end to ensures backward compatability.


Add gov proposal message to ADD/UPDATE/REMOVE required attributes for a restricted marker.

Summary

Add support to change the required attributes on a restricted marker via gov proposal.
Problem Definition

Proposal

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Sure, gov proposal is a useful way to do this ... but someone with transfer granted to them should be able to edit these values at anytime...

Good point.  I will modify this issue to include that.


WorkspaceEdit.insert used to support inserting new lines but not anymore

Version: 1.49.0-insider
Commit: 4ccab0deda29628076ce1bbb17e9fef239337f21
Date: 2020-08-25T08:14:36.185Z
Electron: 9.2.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 18.7.0

Steps to Reproduce:
let file = Uri.file('path/to/file.txt')
let edit = await new WorkspaceEdit()
edit.createFile(file)
edit.insert(file, new Position(0, 0), "test\n\ncreate new line\n\ninsert")

Does this issue occur when all extensions are disabled?: Yes

works for me. tho, you must apply the workspace edit, you makes no sense to await the new-call. My working snippet looks like this
let file = vscode.Uri.file('/Users/jrieken/Code/_samples/devfest/issues/105378/test.txt')
		let edit = new vscode.WorkspaceEdit()
		edit.createFile(file)
		edit.insert(file, new vscode.Position(0, 0), "test\n\ncreate new line\n\ninsert")
		const success = await vscode.workspace.applyEdit(edit);
		console.log(success)

works for me. tho, you must apply the workspace edit, you makes no sense to await the new-call

i reinstalled vscode and it started working, so sorry for the inconvenience & many thanks for the tip 🏆


Support installing a mix of pre-release and normal extensions from CLI

At present CLI supports --pre-release flag that allows installing pre release version for all passed extensions. It would be useful and efficient to handle installing mix of pre-release and normal extensions in one go.
Eg: code --install-extension pub1.name1@prerelease --install-extension pub2.name2
installs pre release version of pub1.name1 and normal version of pub2.name2

CC @chrmarti

Thanks @sandy081. I'm updating the regex in the devcontainer.json schema to allow for @prerelease.

To verify:

From CLI, install a mix of prerelease and released extensions. For example: code-insiders --install-extension GitHub.vscode-pull-request-github@prerelease --install-extension GitHub.remotehub - this shall install pre-release version of GitHub.vscode-pull-request-github extension and normal version of GitHub.remotehub extension


Support for all the shields from shields.io

I would love to use this package in my module called mos. My module can add badges to README files automatically. I believe I can remove the logic of generating the badges from my module and use this module for that purpose.
What I would need in the shields module is:

[ ] support for all the shields at shields.io
[ ] the ability to specify the style of the shields

I can work on it and create a pull request if your OK with these new features being part of the shields module.

@zkochan Sound good!


TPC-DS query processing should fail if output is empty

Count the number of output records with Beam metrics and fail a query if it's not greater than 0.
Fixes #29198

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

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

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

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.


first and foremost, my goals was to lazy load this app

as much as I can. at this time, I believe this is a performant choice.
here I tried to create just the homepage module
but i just went all in and tried to separate most
of what i need to do..
first of all, i needed a shared module that can be used between modules. this way i can still do lazy loading, but still build reusable code
while creating routes for my homepage, i took this time to setup a pagenotfound error page, then utilized my shared module instead of creating it under the app
the app will not be minimal as possible, this serves as the backbone of the application. page not found shouldn't be a part of the backbone, form me its just an addon to for user experience.
lastly, I wanted to explore creating navlinks but i don't want to wire it up statically, so i used the router service, pulling the routes i created earlier and I can now use that to dynamically generate Top level links to my navigation

overrided main. seems that i really need to only allow PRs to it


Fix preferences default

@greens do You know if anyone is working on fixing those failing tests ? (not related to changes in this PR)
Unit test failures in /home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-thermostat/src/test/test_qubino_flush_thermostat.lua:     Setup Mode should be changed after changing updating preferences Unit test failures in /home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-thermostat/src/test/test_sinope_thermostat.lua:     Check all preferences via infoChanged Unit test failures in /home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-lock/src/test/test_zwave_lock.lua:     When the device is added it should be set up and start reading codes     Code set reports should be handled     Max user code number report should be handled     Reloading all codes of an unconfigured lock should generate correct attribute checks     Setting a user code should result in the named code changed event firing     Setting a user code name should be handled     Master code programming event should be handled     The lock reporting a single code has been set should be handled     The lock reporting a code has been deleted should be handled     The lock reporting that all codes have been deleted should be handled     Getting all lock codes should advance as expected     Lock alarm reporting should be handled     Setting a user code should result in the named code changed event firing when notified via Notification CC Unit test failures in /home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-lock/src/test/test_schlage_lock.lua:     Setting a user code should result in the named code changed event firing     Reload all codes should complete as expected Unit test failures in /home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-lock/src/test/test_samsung_lock.lua:     Setting a user code name should be handled     All user codes should be reported as deleted upon changing Master Code Unit test failures in /home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zwave-lock/src/test/test_zwave_lock_code_migration.lua:     Device added data lock codes population     Device added without data should function     Device init after added shouldn't change the datastores     Device init after added shouldn't change the datastores Unit test failures in /home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-lock/src/test/test_zigbee_lock_code_migration.lua:     Device added data lock codes population     Device init after added shouldn't change the datastores     Device init with new data should populate fields Error: Process completed with exit code 1.


Add SymbolInformation.enclosing_symbol

Towards https://github.com/sourcegraph/scip/issues/154
Previously, it was not possible to determine the enclosing symbol of a local because the syntax for local symbols has no hierarchy (unlike global symbols). This PR closes the gap by adding a field SymbolInformation.enclosing_symbol to allow documenting the "parent" or "owner" of a local symbol.
Test plan
Green CI.

cc/ @donsbot this PR implements the last remaining proposal from https://github.com/sourcegraph/scip/issues/154

The Haskell bindings also need to be updated.

The Haskell bindings are now automatically updated \o/


use Autocomplete-plus

I guess Autocomplete-plus use completions in this package automatically.
If we implement provider of autocomplete-plus, we can provide autocomplete depend on context very nicely.
But It looks like pretty complex. So, I will try to implement `provider, but it takes some time.

+1

+1

@outsideris just to follow up, any progress on this?

@ilanbiala
I was pretty busy. I am going to try implement for supporting autocomplete-plus this month, but implementing its provider is pretty hard.

Hi @outsideris , any progress on this?


Repair Curve pool Grafana chart(s)

OETH and rETH were reversed. I changed the labels in grafana and added a comment in the prometheus repo. This way we don't lose the data we already have.


[Question] I’m having issues performing a hash recheck on an incomplete file

Hello!
Ive  been working with this library a lot lately, it’s very cool and I’d like to learn more about how it works so I can contribute and build some tools with it!
I am having issues performing a hash re-check on partially complete torrent files. This is a process I use:

download any torrent to 50% completion using QBittorrent or similar software
Move the ‘.torrent’ file to monotorrent
Move the 50% complete file to the ‘SavePath’ in MonoTorrent
Perform a re-hash
When the re-hash completes the torrent within MonoTorrent shows 0 percent. I would expect it to show as 50%

If I repeat the above steps using a torrent which was downloaded to 100% using Qbittorrent or similar, Monotorrent processes that fine and will mark the torrent as 100% complete and begin to seed.
Am I doing something wrong with the lack of “FastResume” files? I don’t want to say this is a bug, I don’t know if this is operating as “expected” or not.
—
my goal is to write a software which migrates torrents from Qbittorrent and other clients to MonoTorrent, so thus partial hash recheck is an important piece of the workflow
thanks all!

I have tested this again using Deluge then adding the torrent to MonoTorrent and checking the progress. It seems to work in this case.
However, when i used QBittorrent, then added to MonoTorrent it did not seem to work.  Also, it seems that if:

I download a file to 50% with Deluge, for example: C:/downloads/deluge
I add the torrent to MonoTorrent with a different download directory for example: C:/downloads/monotorrent
I download the file 5% using MonoTorrent
Then I set the directory of MonoTorrent to  C:/downloads/deluge
Then I perform a ReCheck hash using MonoTorroent
MonoTorrents seems to have issues recognizing the file is at 50%
I must remove the torrent from monoTorrent completely, then re-add the torrent with C:/downloads/deluge and perform a 'Recheck hash', under this condition MonoTorrorrent sees the file at 50% completion

Is anyone else experiencing this?
I opened a ticket #515 to better understand how 'ReHashing' works so that I can understand why and when this occurrs.
Thanks all

Is this still an issue? I suspect the underlying problem is that the paths passed to MonoTorrent result in it looking for the partial file in a different location.
While it's a long time since this was filed, can you check the values of TorrentManager.Files[xxx].FullPath and ensure it's actually pointing at the partial file?
I'll close this for now as it's probably just a path issue and there was some conversation on the other ticket :)


⚠ forum.minetest.net has degraded performance

In 21a6aca, forum.minetest.net (https://forum.minetest.net) experienced degraded performance:

HTTP code: 200
Response time: 2190 ms

Resolved: forum.minetest.net performance has improved in d198893 after 29 minutes.


Feature/typed query arguments

Request arguments has been strongly typed. It allows auto documentation of api and use strongly typed object instead of dictionary.

Checklist of what to do:

[ ] Code styling (name of struct, functions, e.t.c)
[ ] Documents styling (.md)
[ ] Review and correction of all issues (Architecture)
[ ] Test how it works manually

Implemented in feature branch


Web cmdlets: support compression

I have noticed that there currently appears to be no support for requesting a compressed response (gzip/deflate) with the Invoke-[RestMethod|WebRequest] cmdlets. It would be useful to have support for this (curl implements this via the --compressed flag).
Conditional support can be added via the AutomaticDecompression property of the WebRequest class for FullCLR, and using a new HttpClientHandler with HttpClient for CoreCLR. Support would be added with a new cmdlet parameter (suggest -Compressed ?)
I'm happy to put a pull request in for this - however just making sure this functionality is desirable before adding.

The web cmdlets should support gzip compressed requests.
Following commands work in PS5/6 on Windows:
Invoke-RestMethod httpbin.org/gzip

gzipped headers
------- -------
   True @{Host=httpbin.org; User-Agent=Mozilla/5.0 (Windows NT; Windows NT 10.0; en-US) WindowsPowerShell/5.1.14393....

(Invoke-WebRequest httpbin.org/gzip).Content

{
  "gzipped": true,
  "headers": {
    "Host": "httpbin.org",
    "User-Agent": "Mozilla/5.0 (Windows NT; Windows NT 10.0; en-US) WindowsPowerShell/5.1.14393.693"
  },
  "method": "GET",
  "origin": "REDACTED"
}

However deflate is not supported:
Invoke-RestMethod httpbin.org/deflate
Invoke-WebRequest httpbin.org/deflate

PS5 errors
Invoke-RestMethod : Block length does not match with its complement.
At line:1 char:1
+ Invoke-RestMethod httpbin.org/deflate
+  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], InvalidDataException
    + FullyQualifiedErrorId : System.IO.InvalidDataException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

Invoke-WebRequest : Block length does not match with its complement.
At line:1 char:1
+ Invoke-WebRequest httpbin.org/deflate
+  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-WebRequest], InvalidDataException
    + FullyQualifiedErrorId : System.IO.InvalidDataException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand

PS6 errors
Invoke-RestMethod : The read operation failed, see inner exception.
At line:1 char:1
+ Invoke-RestMethod httpbin.org/deflate
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], IOExcepti
   on
    + FullyQualifiedErrorId : System.IO.IOException,Microsoft.PowerShell.Comma
   nds.InvokeRestMethodCommand

Invoke-WebRequest : The read operation failed, see inner exception.
At line:1 char:1
+ Invoke-WebRequest httpbin.org/deflate
+  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-WebRequest], IOExcepti
   on
    + FullyQualifiedErrorId : System.IO.IOException,Microsoft.PowerShell.Comma
   nds.InvokeWebRequestCommand

I just verified, that PS5/6 indeed doesn't send Accept-Encoding header. My initial test with Requestb.in was stating wrong that Accept-Encoding: gzip is included in the header. Seems to be a bug on Requestb.in side.
Another test with http://xhaus.com/headers and http://www.reliply.org/tools/requestheaders.php showed that Accept-Encoding is missing.
Furthermore http://httpbin.org/gzip always delivers gzip compressed data, regardless if the header is included or not. Another strange behavior in my opinion.
Anyway, what seems to work is:
Invoke-WebRequest www.google.com -Headers @{"Accept-Encoding"="gzip"}

This returns a gzip compressed version of the website.
I would simply suggest to always include the Accept-Encoding header for all supported formats without the need of a specific parameter. If the client (PowerShell) and the server both support the compression they should use it transparently.

So to clarify, you verified that we do already decompress gzip'd content? You just want to default to requesting compressed content in the header?
Sounds right to me, but I don't know enough to know if there's a downside to that default. Only thing I can think of is a little extra server-side compute, but it's probably worth the bandwidth tradeoff. And certainly none of that should affect the client, right?

Yes, gzip and even deflate response streams are supported!
Streamhelper.cs Line 472

#if CORECLR

        internal static Stream GetResponseStream(HttpResponseMessage response)

        {

            Stream responseStream = response.Content.ReadAsStreamAsync().GetAwaiter().GetResult();

            var contentEncoding = response.Content.Headers.ContentEncoding;

            // HttpClient by default will automatically decompress GZip and Deflate content.

            // We keep this decompression logic here just in case.

            if (contentEncoding != null && contentEncoding.Count > 0)

            {

                if (contentEncoding.Contains("gzip"))

                {

                    responseStream = new GZipStream(responseStream, CompressionMode.Decompress);

                }

                else if (contentEncoding.Contains("deflate"))

                {

                    responseStream = new DeflateStream(responseStream, CompressionMode.Decompress);

                }

            }

            return responseStream;

        }

#else

        internal static Stream GetResponseStream(WebResponse response)

        {

            Stream responseStream = response.GetResponseStream();

            // See if it had a content-encoding, wrap in a decoding stream if so.

            string contentEncoding = response.Headers["Content-Encoding"];

            if (contentEncoding != null)

            {

                if (contentEncoding.IndexOf("gzip", StringComparison.OrdinalIgnoreCase) >= 0)

                {

                    responseStream = new GZipStream(responseStream, CompressionMode.Decompress);

                }

                else if (contentEncoding.IndexOf("deflate", StringComparison.OrdinalIgnoreCase) >= 0)

                {

                    responseStream = new DeflateStream(responseStream, CompressionMode.Decompress);

                }

            }

            return responseStream;

        }

#endif

Honestly I have no idea, why the deflate decompression doesn't work, since I don't have a Debug setup here on my laptop.
Regarding the default headers: Here is the W3 RFC
14.3 Accept-Encoding
The Accept-Encoding request-header field is similar to Accept, but restricts the content-codings (section 3.5) that are acceptable in the response.
       Accept-Encoding  = "Accept-Encoding" ":"
                          1#( codings [ ";" "q" "=" qvalue ] )
       codings          = ( content-coding | "*" )
Examples of its use are:
       Accept-Encoding: compress, gzip
       Accept-Encoding:
       Accept-Encoding: *
       Accept-Encoding: compress;q=0.5, gzip;q=1.0
       Accept-Encoding: gzip;q=1.0, identity; q=0.5, *;q=0
A server tests whether a content-coding is acceptable, according to an Accept-Encoding field, using these rules:
      1. If the content-coding is one of the content-codings listed in
         the Accept-Encoding field, then it is acceptable, unless it is
         accompanied by a qvalue of 0. (As defined in section 3.9, a
         qvalue of 0 means "not acceptable.")
      2. The special "*" symbol in an Accept-Encoding field matches any
         available content-coding not explicitly listed in the header
         field.
      3. If multiple content-codings are acceptable, then the acceptable
         content-coding with the highest non-zero qvalue is preferred.
      4. The "identity" content-coding is always acceptable, unless
         specifically refused because the Accept-Encoding field includes
         "identity;q=0", or because the field includes "*;q=0" and does
         not explicitly include the "identity" content-coding. If the
         Accept-Encoding field-value is empty, then only the "identity"
         encoding is acceptable.
If an Accept-Encoding field is present in a request, and if the server cannot send a response which is acceptable according to the Accept-Encoding header, then the server SHOULD send an error response with the 406 (Not Acceptable) status code.
If no Accept-Encoding field is present in a request, the server MAY assume that the client will accept any content coding. In this case, if "identity" is one of the available content-codings, then the server SHOULD use the "identity" content-coding, unless it has additional information that a different content-coding is meaningful to the client.
      Note: If the request does not include an Accept-Encoding field,
      and if the "identity" content-coding is unavailable, then
      content-codings commonly understood by HTTP/1.0 clients (i.e.,
      "gzip" and "compress") are preferred; some older clients
      improperly display messages sent with other content-codings.  The
      server might also make this decision based on information about
      the particular user-agent or client.
      Note: Most HTTP/1.0 applications do not recognize or obey qvalues
      associated with content-codings. This means that qvalues will not
      work and are not permitted with x-gzip or x-compress.

Maybe we should open an issue fixing the deflate decompression bug and another issue for the default headers?

I wrongly presumed this to not support automatic decompression (was looking at reference source) - thanks for clarifying @ThreeFive-O
I will take a look at the deflate issue later, think it's definitely worth a new git issue

I've been able to set up a Debug environment to search for the deflate decompression problem.
Fun fact: The above StreamHelper class I referenced is totally unnecessary and always gets skipped by the code, since the contentEncoding variable doesn't contain gzip or deflate.
Found this instead in WebRequestPSCmdlet.CoreClr.cs for CoreCLR:
        internal virtual HttpClient GetHttpClient()
        {
            // By default the HttpClientHandler will automatically decompress GZip and Deflate content

Hence, there is no need for the StreamHelper class to check for gzip or deflate, since the HttpClient for CoreCLR should do it automatically. On FullCLR the WebRequest or HttpWebRequest class probably does the same thing (not verified).
I guess the problem with deflate decompression is on those .NET classes.
Any ideas how to tackle the problem?
According to #3046 there are many bug fixes planned for the web cmdlets. Maybe some cleanup of the code would be good, if all those problems are getting fixed.

@ThreeFive-O HttpWebRequest is still not in CoreCLR.

Well, I couln't find any information on the .NET Standard where it is supported. I only checked the master branch of CoreFX: https://github.com/dotnet/corefx/blob/master/src/System.Net.Requests/src/System/Net/HttpWebRequest.cs
Maybe as part of .NET Standard 2.0? I guess PS is on 1.6?

Our plan is to move  to .NET Core vNext that supports .NET Standard 2.0 #3046
.Net Core RoadMap https://github.com/dotnet/core/blob/master/roadmap.md

@iSazonov Thank you for clarification. Should I open an issue for moving from HttpClient to HttpWebRequest? Or should I wait until PS moved to .NET vNext?
@Lee303 Would you please submit a PR which fixes the Accept-Encoding problem with the solution you initially stated (AutomaticDecompression and HttpClientHandler)? I'm glad to assist you debug the problem, why HttpClientHanlder doesn't send the correct header by default as stated in the source comment. Do you also want to open a separate issue for the deflate decompression problem or do you want to fix it under this issue?

Just found that deflate problem is already known for Invoke-WebRequest #1753

.Net Core new version release date is very optimistic. I believe that we should wait.
It would be useful to know the opinion of Powershell team.
/cc @SteveL-MSFT

I don't think this capability blocks any scenarios related to cloud management that I'm aware of, so this isn't something that would block the 6.0.0 release, hence we wait.

@SteveL-MSFT Maybe we should mark such Issues by special tag Net20-Waiting ?

@PowerShell/powershell-committee tagged this to discuss whether we want to have a new label since our current plan is to align with .Net Standard 2.0

@SteveL-MSFT Candidates for the new label #3194 #3172

@PowerShell/powershell-committee discussed this and we will create a label called Waiting - NetStandard20

From #1753: TFS:7976639

It looks like there's been some movement here. When issuing this command in PS6:
Invoke-RestMethod https://httpbin.org/deflate

You get this error now:
Invoke-RestMethod : The archive entry was compressed using an unsupported compression method.
At line:1 char:1
+ Invoke-RestMethod https://httpbin.org/deflate -Headers @{'accept-encoding' =  ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], InvalidDataException
+ FullyQualifiedErrorId : System.IO.InvalidDataException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

Interestingly enough, if you try Brotli:
Invoke-RestMethod https://httpbin.org/brotli

IRM doesn't throw an error, but returns the raw Brotli-encoded response:
║��mF╚��>�&����0�� ~Q,����<�����̳���t�d
                                      L�w5�╔|�=�~38�B��O��X6��I�D�<�{!�)��r�UI�       +D╚>������H*$��'
                                                                                8

Seems like a decent workaround in the meantime is to manually send a Accept-Encoding header with gzip and identity every request:
Invoke-RestMethod https://httpbin.org/gzip -Headers @{'accept-encoding' = 'gzip'}

Brotli is not yet supported in HttpClient and thus not in the web cmdlets.
Deflate, however, is supported... but only per RFC. the reality is many sites (httpbin.org included) use a non-rfc compression for deflate.. and there isn't much we can do about that. If DeflateStream could support automatic detection and deflation that would solve our problems, but there's not much we can do at the higher level abstraction where the web cmdlets sit.

Would it make sense to have any response that is brotli encoded throw the same System.IO.InvalidDataException that deflate throws? At least until brotli is natively supported in HttpClient?

Can someone explain the difference between -Headers @{"Accept-Encoding"="gzip"} (that works with our server) and -TransferEncoding GZip (that doesn't, yielding 400 Bad Request)?

I think the compression is supported by new HttpClient from .Net 3+.

@sba923 commented on Jul 10, 2020

Can someone explain the difference between -Headers @{"Accept-Encoding"="gzip"} (that works with our server) and -TransferEncoding GZip (that doesn't, yielding 400 Bad Request)?

Accept-Encoding and Transfer-Encoding are two different headers:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Accept-Encoding
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Transfer-Encoding

The Accept-Encoding request HTTP header indicates the content encoding (usually a compression algorithm) that the client can understand. The server uses content negotiation to select one of the proposal and informs the client of that choice with the Content-Encoding response header.

Transfer-Encoding is a hop-by-hop header, that is applied to a message between two nodes, not to a resource itself. Each segment of a multi-node connection can use different Transfer-Encoding values. If you want to compress data over the whole connection, use the end-to-end Content-Encoding header instead.

This answer on StackOverflow goes into the differences between Content-Encoding (which is related to Accept-Encoding) and Transfer-Encoding a bit more.

I think this issue was fixed by #17955


Acceptance tests should use bolt's internal puppetfile

It looks like the acceptance tests preempt bolt's bundled content by managing modules separately acceptance/setup/common/pre-suite/071_install_modules.rb. This is both unrealistic and requires us to maintain two sources of truth. Instead of installing these modules as a presuite each install method should manage these modules if needed from bolts internal Puppetfile.

Oh, oops I thought you wanted that as just a maint commit. https://github.com/puppetlabs/bolt/pull/1346 I raised that PR before I saw this. Can fix it up with ref to this issue.


Action buttons (refund/reject/download receipt) missing from transactions page

Describe the bug
When loaded directly, the transactions page is missing the action buttons in "View details".
To Reproduce
Steps to reproduce the behavior:

Go to the transactions page for a collective where you have special permission, e.g. your personal profile
Click on 'View details' for one of the transactions
See missing buttons

Screenshots

Context
Seems to come from https://github.com/opencollective/opencollective-frontend/pull/8751/files#diff-03f599cf73f53c2bdce53bde29fc9f1e4f34ce47e9d386c1720511a2673c981dL250 where we removed the logic to refetch when users log in.

Related: this button is also missing because we don't refetch permissions and because the way we check the account type in https://github.com/opencollective/opencollective-frontend/blob/3f826b1939175f660846e8f174ba65e16979dae2/components/transactions/TransactionsPage.tsx#L126 was not compatible with data from V2.