Plugins stretches/rotates images with certain EXIF settings

Not sure if this is jut me, but when using the focus plugin, uploaded images with certain EXIF rotation settings get rotated/stretched.
Step to reproduce

Download EXIF orientation sample images from this repo: https://github.com/recurser/exif-orientation-examples
Download and setup Kirby Starterkit: https://github.com/getkirby/starterkit
Login to the panel and navigate to any album (e.g. Photograph->Trees)
Upload the sample photos, everything should work as expected
Download and install the Focus plugin
Delete the sample photos or navigate to another album
Upload the sample photos again, numbers 5-8 will be stretched and/or rotated in the wrong way

I fixed this with a hook in the meantime.

First of all, thanks for this excellent description!
I went through the steps and ended up at no. 4 because the photos 5-8 already get stretched on a fresh starterkit installation without any additional plugins – at least for me. Pleaser find attached the screenshots from the Panel.
Starterkit 3.3.5
ImageMagick 7.0.10-6_1
The ImageMagick class used by the Focus plugin is almost the same as the one used by Kirby. So this issue might be Kirby related, @bastianallgeier!?

Ah, interesting! I first thought it might be a Kirby issue as well.
Have you tried if the auto-orient setting for thumbs has any effect? It didn’t for me, unfortunately.

Auto-orient had no effect for me, too.
I close the issue for now as it seem to be Kirby related. If you run into any issues with this plugin again, don't hesitate to contact me again!


Make --port compatible with TLS terminating reverse proxies

Disables TLS handling on the server if the route is TLS but --port was given, to make it so that ob run --port can be used with reverse proxies that terminate TLS.
I have:

[x] Based work on latest develop branch
[x] Followed the contribution guide
[ ] Looked for lint in my changes with hlint . (lint found code you did not write can be left alone)
[x] Run the test suite: $(nix-build -A selftest --no-out-link)
[x] Updated the changelog
[ ] (Optional) Run CI tests locally: nix-build release.nix -A build.x86_64-linux --no-out-link (or x86_64-darwin on macOS)

Thanks @plt-amy !!


Add allthedoll's taco recipe

This pull request add's @allthedoll's taco recipe and closes Jira ticket 1234.
cc @organization/team-name for visibility

Hey @allthedoll, this is a great recipe! ✨ What if we maybe added black olives?


First hack at upgrade script for GCE

Address #6075: Shoot the master VM while saving the master-pd. This
takes a couple of minor changes to configure-vm.sh, some of which also
would be necessary for reboot. In particular, I changed it so that the
kube-token instance metadata is no longer required after inception;
instead, we mount the master-pd and see if we've already created the
known tokens file before blocking on the instance metadata.
Also partially addresses #6099 in bash by refactoring the kube-push
path.

cc @mbforbes

Thanks, Zach. Looking good mod comments; ping me when you've addressed them. Will be great to get to start testing this out (& I can hack on the nodes part as soon as this is in).

PTAL. I think I addressed your comments, except the refactoring of the master init and the bangs.

I played around with that refactor and agree it looks fine. So, yes, TOTAL PTAL.

(I kept it on a separate commit for easier reviewability, though.)

LGTM


fix: adjust button positioning for Firefox compatibility

Description
Refactored the NewsletterForm component by changing the width of the input field to 44 units. This ensures a consistent and visually appealing layout on both Chrome and Firefox.

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

Fixes #1729
Mobile & Desktop Screenshots/Recordings

Before:

After:

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Hey rita, here is how it looks on firefox... A little cut on the input edge

Hey rita, here is how it looks on firefox... A little cut on the input edge

Thank you for the review, @OgDev-01. My latest commit has fixed the above concern.

Looks like in a smaller window it still gets chopped off (even on Chrome):

Thank you for the review. I have done the fix and tested it on Chrome, Firefox, and Safari browsers; it works fine now.


mWeb Safari - Workspace chat tooltip not positioned correctly

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.74-8
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @rayane-djouah
Slack conversation (hyperlinked to channel name): expensify_bugs
Action Performed:

Log in as User A, an account that owns a workspace
Create a new account B
On A, Invite B to the workspace
On B, open the workspace chat
Verify a tooltip shows over the composer's Create button saying "Get started! Submit your first expense"

Expected Result:
The tooltip should be positioned over the composer's Create button on mWeb Safari same with other platforms
Actual Result:
The tooltip is not correctly positioned on mWeb Safari unlike other platforms
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
WS chat tooltip in mobile Safari is positioned incorrectly.
What is the root cause of that problem?
We calculate the position of the tooltip based on the anchor y offset and the tooltip height. The calculation here is already correct.
https://github.com/Expensify/App/blob/082e14b9bc1259c329c7cd186ee8f21c0397f6fb/src/styles/utils/generators/TooltipStyleUtils/index.ts#L159
However, on iOS, the viewport is scrolled when the keyboard is shown. The viewport scroll explanation and visualization are well explained here: https://github.com/Expensify/App/issues/16082.
Let's use an example. On my simulator, the composer yOffset without keyboard is 666px.

When the keyboard shows, the composer slides up which is closer to the top of the screen, so the yOffset is now 388px.

Let's ignore the tooltip height from the calculation and only use the composer yOffset as the top position of the tooltip. Since top is positioned based on the viewport, it won't be placed correctly above the composer.

What changes do you think we should make in order to solve the problem?

We need to account for the viewport scroll amount for the top calculation. Tooltip has a shift vertical which we can use so we only changes this for WS chat tooltip.
https://github.com/Expensify/App/blob/082e14b9bc1259c329c7cd186ee8f21c0397f6fb/src/styles/utils/generators/TooltipStyleUtils/index.ts#L159
https://github.com/Expensify/App/blob/082e14b9bc1259c329c7cd186ee8f21c0397f6fb/src/pages/home/report/ReportActionCompose/ReportActionCompose.tsx#L461-L462
const offsetTop = useViewportOffsetTop();

shiftVertical={variables.composerTooltipShiftVertical + offsetTop}

(or we can apply it to measureTooltipCoordinate so it's applied to all educational tooltip)
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?

N/A
What alternative solutions did you explore? (Optional)
Or we can use measureLayout and measure it relative to document.body.
https://github.com/Expensify/App/blob/082e14b9bc1259c329c7cd186ee8f21c0397f6fb/src/components/Tooltip/EducationalTooltip/measureTooltipCoordinate/index.ts#L4-L9
return target?.measureLayout(document.body, (x, y, width, height) => {
    updateTargetBounds({height, width, x, y});
    showTooltip();
});

but measureLayout isn't available on the RN 0.76 docs anymore, so not sure if it's safe to use it.

This is buggy looking, seeing as it covers up workspace information on the page I think this needs to be fixed.

@bernhardoj How do you enable EducationalTooltip when keyboard opens?

@hungvu193 you need to open an empty chat, so the composer is focused automatically.

@hungvu193 you need to open an empty chat, so the composer is focused automatically. (I manually change the show condition to true too)

I did the same thing. But on mSafari I didn't see the composer got focused on opening

Thanks @bernhardoj. Your proposal looks good to me!
🎀 👀 🎀  C+ reviewed


Get single Docker instance of DPP UI working on CEREUS

Task to do
Get a single Docker instance of the UI working on CEREUS
Reason
Allow users to run workflows
Result
Users are able to run workflows on Cereus

Waiting on VPN permissions

Have VPN access and  can run a job: http://gpu06.cyverse.org:5000/


Update master to build 3.0 preview submodules

This updates the submodules to what was used for the 3.0 preview Microsoft build.
Some cleanup I have in mind for future PRs:

Stop smoke-test replacing netcoreapp2.2 with netcoreapp3.0, originally there to work around the templates being old. Running dotnet new console with the source-built SDK, for example, creates a csproj with netcoreapp3.0.
Look through the prebuilts this PR to try to nip them in the bud.
Merge patches into repos. Some patches might already have PRs out or be merged, but we can't use the merged commits yet because they aren't 3.0 preview.
Look for patches that could be replaced with different build args or a pre-build modification.

CI error seems really weird, still working on it. The first project (C# console) hits an error:
    running run
/work/testing-smoke/builtCli/sdk/3.0.100-preview-009812/Roslyn/Microsoft.CSharp.Core.targets(52,5):
  error : Roslyn compiler server reports different hash version than build task.
  [/work/testing-smoke/C#_console/C#_console.csproj]

I'm able to repro with this command:
docker run --rm -it \
  -v /home/dagood/sb/source-build:/work:z -w /work \
  microsoft/dotnet-buildtools-prereqs:centos-7-b46d863-20180719033416 \
  ./smoke-test.sh --projectOutput

but not when I add -u=1000:1000 to become not root:
docker run --rm -it -u=1000:1000 \
  -v /home/dagood/sb/source-build:/work:z -w /work \
  microsoft/dotnet-buildtools-prereqs:centos-7-b46d863-20180719033416 \
  ./smoke-test.sh --projectOutput

(The user thing seemed to be a red herring. My guess would be that for some reason, Roslyn decided not to use the compiler server at all in that case.)
I tracked this down to this new feature: https://github.com/dotnet/roslyn/pull/29711, specifically here:
// early check for the build hash. If we can't find it something is wrong; no point even trying to go to the server
if (string.IsNullOrWhiteSpace(BuildProtocolConstants.GetCommitHash()))
{
    return new IncorrectHashBuildResponse();
}

That's based on the assembly's CommitHashAttribute, which ilspy tells me is [assembly: CommitHash("")] in our case.
Roslyn has a target that generates the attribute here: https://github.com/dotnet/roslyn/blob/d7e493956ffcbf0e932097f6f3e1fa3fa1699087/eng/targets/Imports.targets#L94. We set OfficialBuild to true, but we don't provide a SourceRevisionId so it stays empty.
It looks like SourceRevisionId normally comes from sourcelink, which we don't use: https://github.com/dotnet/sourcelink/blob/master/docs/README.md#sourcerevisionid
I'm going to try setting SourceRevisionId to the GitCommitHash we have saved.

Now we get a "this should never happen" error on two legs of CI--one online tarball build and one offline tarball build. Seems like flakiness, maybe race condition? It doesn't repro for me locally. I don't see any way to drill into what actually failed to happen (especially without a local repro). Message:
[SourceBuiltSdkResolver] Looking for SDK Microsoft.Build.CentralPackageVersions. Detected config(s) in env: 'REPO_TOOLSET' for 'RoslynTools.RepoToolset/1.0.0-source-built' at '/tb/tarball_57394/Tools/source-built/RoslynTools.RepoToolset/sdk/'
/tb/tarball_57394/src/msbuild/Directory.Build.targets(14,3): error : [SourceBuiltSdkResolver] 0 overrides found for 'Microsoft.Build.CentralPackageVersions'
/tb/tarball_57394/src/msbuild/Directory.Build.targets(14,3): error : Failed to resolve SDK 'Microsoft.Build.CentralPackageVersions'. Package restore was successful but a package with the ID of "Microsoft.Build.CentralPackageVersions" was not installed.
/tb/tarball_57394/src/msbuild/Directory.Build.targets(14,33): error MSB4236: The SDK 'Microsoft.Build.CentralPackageVersions' specified could not be found. [/tb/tarball_57394/src/msbuild/src/Build/Microsoft.Build.csproj]

(The SourceBuiltSdkResolver not finding any overrides is normal, it's just that MSBuild only shows the message when all resolvers fail to acquire the requested SDK.)
Source of the NuGet error message:
// This should never happen because we were told the restore succeeded.
// If we can't find the package from GetAllInstalled(), we probably did something wrong with the NuGet API
errors.Add(string.Format(CultureInfo.CurrentCulture, Strings.PackageWasNotInstalled, sdk, sdk.Name));

Strings.PackageWasNotInstalled is Failed to resolve SDK '{0}'. Package restore was successful but a package with the ID of "{1}" was not installed.
I'm going to work around this by using SourceBuiltSdkResolver to find Microsoft.Build.CentralPackageVersions, bypassing the NuGet resolver entirely. It's a ref-only prebuilt so we should have easy access to it.

The override workaround includes a change to make the resolver detect the sdk directory case insensitively because the CentralPackageVersions package has it in uppercase.


Add GitHub workflow to sync main with master

I would like to propose renaming the main line branch from master to main as part of the move to nrf-rs. If the main line branch is renamed to main then it would be useful to keep master in sync to avoid any compatibility issues.
This requires a Deploy Key for writing to the repo
https://docs.github.com/en/developers/overview/managing-deploy-keys#deploy-keys

Is this ready? I just did the rename. 😬

Not quite, it needs a deploy key to push the changes.

Hm, how about we forget about this for now and revisit if and when it becomes a problem?

I can look at it tomorrow.


API files get undeployed, when the NFS server goes down

Description:
When the /server directory is mounted with NFS, all the APIs are undeployed from the gateway once the NFS Server goes down. This is due to the hot undeployment feature where missing files will be undeployed by running a scheduler task. The two scenarios where a single file gets deleted and the whole directory goes down due to NFS failures should be distinguished and we should stop hot undeployment when the whole directory disappears.
Steps to reproduce:

Mount the /server directory with NFS
Create an API and invoke.
Stop the NFS server and try to invoke the API.

Affected Product Version:
APIM 3.2.0

duplicate https://github.com/wso2/product-apim/issues/7353


[Programmatic Issue - Settings>Image Resizer>File Explorer File]: Name property of a focusable element is set to be null in 'File' dialog.

[Power Toys Settings - Image Resizer > File Explorer(System)]
User Experience:
This will impact the screen reader users if Name property of a focusable element is set to  null in 'File' dialog.
Pre-Requisite:
Accessibility Insights for Windows application should be installed in the system.
Test Environment:
OS Version: 20221.1000
App Name: Power Toy Preview
App Version: v0.23.0
Screen Reader: Narrator
Repro Steps:

Open File Explorer using Win+E key.
Navigate to any photo and press right click.
Navigate to 'Image Resizer' and activate it. Image Resizer will open.
Navigate to Settings link and activate it.
Run Accessibility Insights for Windows and verify the issue on File tab.

Actual Result:
Name property of a focusable element is set to be null in 'File' dialog.
Note:
Same issue is repro in Encoding, File and sizes dialog for all the edit fields.
Expected Result:
Name property of a focusable element should not be be null in 'File' dialog. It should have appropriate name.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={248054a6-5e68-4771-9e1e-242fb5025730}
29_Image Resizer_Mas4.1.2_Name property of a focusable element is set to null in File_details.zip
29_ImageResizer_Mas4.1.2_Name property of a focusable element is set to null in File.zip

Shipped with 0.27 release. https://github.com/microsoft/PowerToys/releases/tag/v0.27.0

UI is updated, Control is no longer available. Verified on OS version: 20257.1, App Version: v0.27.0


Multiple provider blocks getting authentication 10000 error from cloudflare_record resource

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.3.6
on windows_amd64
+ provider registry.terraform.io/cloudflare/cloudflare v3.32.0
+ provider registry.terraform.io/hashicorp/google v4.31.0
+ provider registry.terraform.io/hashicorp/kubernetes v2.12.1

Affected resource(s)
cloudflare_record
Terraform configuration files
resource "cloudflare_record" "app_production" {
  provider = cloudflare.production
  zone_id  = var.cloudflare_config.production_zone_id
  type     = "A"
  name     = "${kubernetes_namespace.environment_namespace.metadata[0].name}.${data.cloudflare_zone.production_zone.name}"
  value    = data.kubernetes_service.nginx.status[0].load_balancer[0].ingress[0].ip
}

resource "cloudflare_record" "app_development" {
  provider = cloudflare.development
  zone_id  = var.cloudflare_config.app_zone_id
  type     = "A"
  name     = "${kubernetes_namespace.environment_namespace.metadata[0].name}.${data.cloudflare_zone.app_zone.name}"
  value    = data.kubernetes_service.nginx.status[0].load_balancer[0].ingress[0].ip
}

data "cloudflare_zone" "app_zone" {
  provider = cloudflare.development
  zone_id  = var.cloudflare_config.app_zone_id
}

data "cloudflare_zone" "production_zone" {
  provider = cloudflare.production
  zone_id  = var.cloudflare_config.production_zone_id
}

variable "cloudflare_config" {
  sensitive   = true
  description = "Cloudflare zone api token"
  type = object({
    development_api_token = string
    production_api_token  = string
    app_zone_id           = string
    production_zone_id    = string
  })
}

terraform {
  required_version = ">= 1.0.0"

  required_providers {
    google = {
      source  = "hashicorp/google"
      version = "4.31.0"
    }

    kubernetes = {
      source  = "hashicorp/kubernetes"
      version = "2.12.1"
    }

    cloudflare = {
      source  = "cloudflare/cloudflare"
      version = "3.32.0"
    }
  }
}

Link to debug output
https://gist.github.com/shanehughes1990/d0d1861722bc62066238bf91ef2c911d
Panic output
No response
Expected output
I would expect that the cloudflare_record be created as it doesn't exist in that zone yet
Actual output
Getting authentication error 10000
Steps to reproduce

Cloudflare Account with 2 zones
Cloudflare provider block for each zone with seperate api_token for each with correct DNS:Edit permissions on that zone
2 Data sources, one for each alias
2 cloudflare_records one for each alias

Additional factoids
The strange part about all of this is I am 100% confident the credentials are correct each provider has a seperate api token with DNS:Edit permissions on there respective zones (and I have other workspaces with only one cloudflare provider and working fine with production api token), and the data resource for the production alias is in state (and correct) but the cloudflare_record for production alias fails with auth error? I would assume the data resource would also fail if that was the case.
Also all cloudflare.development resources are working and deployed just fine.
References
No response

I do believe I figured out what my issue was and was with the cluster issuer itself, it has 3 solvers (all cloudflare) and I broke the cluster issuers into 3 isolate resources and all is working again.


Bug: Tooltip truncated in the left if the host element is near the left of the screen

Bug:
Tooltip displayed at the bottom truncated in the left if the host element is near the left of the screen
What is the expected behavior?
The left margin of the tooptip should be zero or slightly above zero, rather than currently negative.
What is the current behavior?
Characters in the left get truncated.
What are the steps to reproduce?
In https://material.angular.io/components/tooltip/examples with Plunker, make the selection the host get closer to the left through remove the left margin in the css, and the tooltip position set to the bottom, and also make the content of the tooltip slightly longer.
What is the use-case or motivation for changing an existing behavior?
There are some elements on the left need tooltip to be displayed at the bottom of the element.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Material2 Beta 7. All browsers are affected.

@jelbourn will this be addressed by your work on overlay?

Tooltip should already use fallback positions to avoid this problem.

Very annoying issue. Is there a workaround until this is fixed?


Oem kill does not seem to work anymore

Ever since 5.0 or maybe earlier, lauching OTD with OemKill enabled does not stop the Wacom drivers anymore.
Is that my fault Do I need to run the Installer/laucher as admin?
As for now I kill the driver manually in task bar.

This is not an issue for Plugin-Repository and should be moved to VoiDPlugins

Okay posted it there.


Separate, auto-updatable branch for unstable releases

I would like to have separate auto-updatable branch for unstable releases. We would need it for auto updatable docker builds on https://hub.docker.com/. Triggering rebuilds on every commit would be just too much. This would be also very usefull for builds of private projects on private hydra instances.

Now we have https://github.com/NixOS/nixpkgs-channels/branches, so I think your use case is solved.


Stylize, Add Readme & Deploy

Add colors / fonts
Create media (gifs & screenshots)
Finish Readme
Deploy to Heroku & MongoDB Atlas

Integrated under: commit 47131457f4077d1051931e81c408477119a499af .

Integrated under: commit 47131457f4077d1051931e81c408477119a499af .


Project and maintainer status.

Howdy.
We are using python-consul heavily in our environment to interact with our consul cluster and we are getting ready to upgrade to consul 1.x.x. PR https://github.com/cablehead/python-consul/pull/182 seems to address this but it has been lying around for more than 6 months.
So, @abn and @cablehead, how is the development status on the project? Is it dead? At this state we are considering forking or finding another client.

We are actively looking for additional maintainers. If you come across this and have time, interest, and ideally, use python-consul actively at work, please leave a message!

Let me check with work if I can take over in some capacity

@cablehead I think I can help.

@matusvalo Is it possible to shoot me an email? (my email address should be on my GitHub profile)

We also use python-consul at work.
I'm in the same position as @eliasbrange.
Willing to scratch my own itches and push upstream if you take it.
May I ask why https://groups.google.com/forum/#!forum/python-consul and https://groups.google.com/forum/#!forum/python-consul-admin require login for reading?
I usually don't use my google account. Would hate to login just to read a forum. Posting is something different of course.

@asteven no reason. I think I've just changed it so anyone can now read messages (there aren't any :))
I've also deleted the python-consul-admin list. It'll be progress to just get one email list going.

I wonder if it would make sense to join forces with consulate. It seems there is a very large overlap in functionality. The main difference I see from a quick look is that python-consul supports asyncio, tornado, twisted and friends. But if this is the only difference, would it make sense to contribute implementations for that to consulate instead of struggling to maintain 2 projects?
I know it is somewhat blasphemous to mention this in your issue tracker.

Not remotely blasphemous. We aren't guilds :)

supports asyncio, tornado, twisted and friends. But if this is the only difference, [...]

All right... all right... but apart from better sanitation and medicine and education and irrigation and public health and roads and a freshwater system and baths and public order... what have the Romans done for us?

I think you've caught the core difference there. To handle async, you kinda need to plan for it from the start.
Consulate was actually around before python-consul. I haven't track it's development over the years, so things may have change, but this was why I started this module at the time:
https://groups.google.com/d/msg/consul-tool/lUVdUeo5eEM/-mH0qhNUrqQJ
One of the reasons Consulate has such a nice API for exposing Consul's key / value store as a python dict is (at least at the time) it didn't concern itself with async networking. It wasn't possible at the time to even get the index of the previous read, so you could then long poll for changes since that index.
The goal of python-consul is to expose simply and conveniently the full capabilities of Consul's API. It's agnostic to usage after that. I'm hoping that makes it easy to then layer convenience layers for particular use cases on top.
Honestly though, what python-consul does is so simple if you do have a narrow use case, you could probably roll the part python-consul does you in half an afternoon. And I think that's calculation that's worthwhile making for projects.

I'm looking to push a new release to pypi tomorrow afternoon with #190 and #218
Shout out if you'd like to squeeze anything else in! Particularly handling breaking changes from Consul.

Well I have a unbiased interest in #217  :-)
But it probably needs some more eyes and possibly work to ensure it doesn't break anything.
Comments would be much appreciated.

@cablehead hi, what about maintainership?

Hey @cablehead, I'm also using this lib actively and I think I may help to maintain and review some PRs.
Do you still need help?

@cablehead this project is great, need to get those PR's merged :)

I guess we should go to a maintained fork with the most number of stars: https://github.com/poppyred/python-consul2


Create 9. Greedy Algorithms - Kruskal's MST.md #5512

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #issue_number_here

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ ] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[ ] I follow Contributing Guidelines & Code of conduct of this project.
[ ] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My changes generates no new warnings.
[ ] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@Apoorv-cloud Someone else was already working on the issue. It has been told several times don't use the assign command on issues created by supervisors.
You can look for something else to work on.


IoT extension update 0.9.0 + re-branding to azure-iot

This is a re-branding/re-aliasing of the Azure CLI IoT extension. Instead of the old azure-cli-iot-ext the new alias is azure-iot. The old references are not removed yet because we don't want to break our users, and we need time to get various docs and script references updated. We'll get a deprecation timeline in play and remove the old alias after.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/digimaun/azure-cli-extensions.git@iotext090#subdirectory=src/$EXT&egg=$EXT"

@digimaun @qwordy Can you please hold on merging this change? We have an Azure Pipelines task depending on this extension and will be impacted by this change. I'm working to update our task and expected to finish by tomorrow. Thank you very much!

Thanks @yungezz @digimaun @qwordy Please go ahead and merge this change.

suggest to add descriptive information in existing extension to route user to the new extension.

suggest to add descriptive information in existing extension to route user to the new extension.

We were thinking of doing that after this merge in the following elements:

IoT extension github readme with clear visible indicators
Work with the docs team to replace azure-cli-iot-ext with azure-iot in all references
Update of the IoT command module 'try out the extension' event handler (for the base CLI) to point to the new name
Recording a Channel 9 video
In various communication with users/stakeholders

We could update the existing/old extension information but keep in mind its the same code base and project. That would mean a new release of "v0.8.10" with the old alias and the primary change being to the package description (to state its deprecated).
With this PR, the old extension is not going away until the deprecation timeline ends. Because of this, I think we have more time/flexibility to remove the existing alias in a graceful manner. We just want to make sure the new alias is available on the index (first).

thanks for the detail plan sharing.

Bump. @qwordy , please let me know if you need anything else to merge.


Update dev.yml

Install ALL versions of Spark (just include existing spark role for that).
Do some refactoring.
Remove [dev] from inventory file (this causes issues to other playbooks).

@Y4Rv1K, please take a look.


add official PostgreSQL-cheat-sheet #38

fix #38 and reach milestone
added official PostgreSQL-cheat-sheet

It seems that is not possible to see .pdf files here in GitHub, Shall we change it to .md? I can not open the pdf

It seems that is not possible to see .pdf files here in GitHub, Shall we change it to .md? I can not open the pdf

I can see .pdf files. Also you can open the .pdf locally


Error when value of param name contains <

For example
https://ui-avatars.com/api/?name=i love myself <3

Similar issue with &, URL encoding doesn't work because it turns into %


[qt5-base] Build error

Package: qt5-base:x64-osx-dynamic -> 5.15.10#2
Host Environment

Host: x64-osx
Compiler: AppleClang <IP_ADDRESS>00029
vcpkg-tool version: 2023-06-22-f19f3d9939100085fe2f7e41502d60b08e2083b6
vcpkg-scripts version: 228987a6f 2023-07-05 (6 days ago)

To Reproduce
vcpkg install
Failure logs
...

This is the Qt Open Source Edition.

You have already accepted the terms of the Open Source license.
ERROR: debug-only framework builds are not supported. Configure with -no-framework if you want a pure debug build.

Additional context
vcpkg.json
{
  "name": "openms",
  "version-string": "3.0.0",
  "dependencies": [
    "boost",
    "xerces-c",
    "qt5-base",
    "libsvm",
    "qt5-svg",
    "zlib",
    "coin-or-cgl",
    "coin-or-clp",
    "coin-or-cbc",
    "coinutils",
    "eigen3",
    "bzip2"
  ]
}

Duplicate of those prematurely closed issues:
https://github.com/microsoft/vcpkg/issues/23730
https://github.com/microsoft/vcpkg/issues/26406


user is authorized even when its removed from access list

rancher version: master
Steps to reproduce:

Enable Github authentication
Select  Allow members of Projects, plus Authorized Users and Organizations
Add ranchermain to authorization list
login with ranchermain user
with admin remove user ranchermain from authorization list
try to login with ranchermain

result:
user ranchermain is authorized normally

closing as duplicate of https://github.com/rancher/rancher/issues/12031
I realize you took slightly different steps to get there, but its teh same bug


Missing properties for using Button as link

Describe the bug
When using our Button component and setting the href property I expect to be able to set other properties related to the anchor element .
Steps to reproduce the bug

Import Button from eds-core-react
Assign value to href

Expected behavior
Expected properties related to anchor element such as target or rel to be available.
Additional context

Solved.


It just doesnt work

I've tried many times , it worked once a long time ago and then randomly stopped working forever , bepinex  loads in the background but when the game loads it's just plain old gross normal subnautica VR . Im stumped , what do i do . I tried deleting everything , reinstalling and going through the whole process but same issue , what do i do

Same here.
Setup: Win10 + Quest 3 Headset
Have installed Bepines + SumbersedVR
Tried a few times, did not work, normal movement with headset worked
Then I added "-vrmode openvr" to the Subnautica icon in steam.
Sumersed worked perfectly with the Qusest 3 Controllers.
But since then it doesn't.
Bepinex loads but have standard movement only (headset), the Quest 3 controllers do not work.
Tried it with SteamLink and QuestLink. Nothing.
How to fix this?

Can you post logs? Unfortunately I don't have a quest 3 to test myself with. The logs can be found in steamapps\common\Subnautica\BepInEx, when you open the local files by right clicking on the Subnautica in steam.

I am using a Quest 2, not 3, but I have the same issue so I'm putting the logs here in case it helps.
LogOutput.log

[Error  :SubmersedVR] SubmersedVR only supports SteamVR! Make sure to add -vrmode openvr to your games advanded launch options when using an oculus headset.

I have added that line to the advanced options, so I don’t know what else to do

Are you sure you extracted the plugin correctly? The folder  SubmersedVR_0.1.6 shouldnt exist.

I am not sure BepInEx will load dlls directly from the plugins directory you might need to put the dll into a subdirectory.

You're right that I had extracted it incorrectly, but fixing it didn't seem to work. I've attached the logs from that attempt.
LogOutput2.log

From your latest log:
[Error  :SubmersedVR] SubmersedVR only supports SteamVR! Make sure to add -vrmode openvr to your games advanded launch options when using an oculus headset.
Make sure to start the game from steamvr and not the oculus dashboard.

I have been doing that, launching the game through steamvr on my desktop while the quest is linked. My exact process is that I set up the Quest Link, open Steam on computer, launch Subnautica in VR mode (with the -vrmode openvr script in the launch options), and then it automatically is broadcast to my oculus but without motion controls. Is there anything in that process I am doing wrong? Thank you all so much for your help, by the way, I really appreciate it

@Okabintaro I had the same issue, and I just tried your method above. It works like a charm!

@Okabintaro I have the same issue, I'm playing on a quest 3 and launching the game trough quest link using steamvr, I have the -vrmode openvr launch options selected yet bepinex outputs the following on bootup:
[Warning:SubmersedVR] Game was not started in VR. Don't load and apply SubmersedVR patches.
Regardless of how the game is started. I followed your advice to launch the game from within quest link but it doesn't function still.

@Okabintaro I have the same issue, I'm playing on a quest 3 and launching the game trough quest link using steamvr, I have the -vrmode openvr launch options selected yet bepinex outputs the following on bootup:
[Warning:SubmersedVR] Game was not started in VR. Don't load and apply SubmersedVR patches.
Regardless of how the game is started. I followed your advice to launch the game from within quest link but it doesn't function still.

Can you try starting SteamVR first in Quest Link and then from within SteamVR on the dashboard start Subnautica. That should guarantee the game being started in VR.

That's what I've been doing unfortunately:
Connect the headset
Activate Quest link through the headset
Boot up steamvr from the Qlink
Boot up subnautica from Steamvr
it still doesn't detect the game being in VR

Nope, never played before on this account. Could I somehow edit the .dll to not run those checks and apply the patches regardless? Could that fix it?

Here's the bepinex log with some personal info omitted:
[Message:   BepInEx] BepInEx <IP_ADDRESS> - Subnautica (26/05/2022 17:27:04)
[Info   :   BepInEx] Running under Unity v2<IP_ADDRESS>0
[Info   :   BepInEx] CLR runtime version: 4.0.30319.42000
[Info   :   BepInEx] Supports SRE: True
[Info   :   BepInEx] System platform: Bits64, Windows
[Message:   BepInEx] Preloader started
[Info   :   BepInEx] Loaded 1 patcher method from [BepInEx.Preloader <IP_ADDRESS>]
[Info   :   BepInEx] Loaded 1 patcher method from [Tobey.Subnautica.ConfigHandler.Patcher <IP_ADDRESS>]
[Info   :   BepInEx] Loaded 1 patcher method from [Tobey.UnityAudio.Patcher <IP_ADDRESS>]
[Info   :   BepInEx] 3 patcher plugins loaded
[Info   :Legacy Config Handler] Applying override mode: Default
[Info   :   BepInEx] Patching [UnityEngine.CoreModule] with [BepInEx.Chainloader]
[Info   :Unity Audio Patcher] Enabling Unity Audio...
[Message:Unity Audio Patcher] Unity Audio successfully enabled.
[Message:   BepInEx] Preloader finished
[Message:   BepInEx] Chainloader ready
[Message:   BepInEx] Chainloader started
[Info   :   BepInEx] 4 plugins to load
[Info   :   BepInEx] Loading [SubmersedVR 0.1.6]
[Warning:SubmersedVR] Game was not started in VR. Don't load and apply SubmersedVR patches.
[Info   :   BepInEx] Loading [File Tree 1.0.0]
[Info   :   BepInEx] Loading [Subnautica Config Handler 1.0.2]
[Info   :   BepInEx] Loading [Unity Audio Patcher 2.0.2]
[Message:   BepInEx] Chainloader startup complete
[Info   : Unity Log] OpenVR initialized!
[Info   : Unity Log] Created eye textures with a "separate" layout.  The "multi-pass" stereo mode will be used.

[Info   : Unity Log] AddressablesUtility.LoadSceneAsync loading scene: StartScreen in mode Single
[Info   : Unity Log] Initializing Steam services
[Info   : Unity Log] OnQuit behaviour queued for deferred registering, platform services aren't initialized.
[Warning: Unity Log] DontDestroyOnLoad only works for root GameObjects or components on root GameObjects.
[Warning: Unity Log] Can't set camera projection to orthographic when XR is running and rendering to the main view. Camera is no longer valid to use for rendering. Please check camera projection settings of all camera components in your scene.
[Info   : Unity Log] AddressablesUtility.LoadSceneAsync loading scene: MenuEnvironment in mode Additive
[Info   : Unity Log] Telemetry session started. Platform: 'Steam', UserId: '76561197960267366', SessionId: 392595295
[Info   : Unity Log] Loading game settings
[Info   : Unity Log] AddressablesUtility.LoadSceneAsync loading scene: XMenu in mode Additive
[Info   : Unity Log] changeset 72550, QualityLevel = 2
[Info   : Unity Log] OK Running Win64
[Info   : Unity Log] Application version: 1.21.72550
[Info   : Unity Log] SystemInfo:
 deviceModel = To Be Filled By O.E.M. (To Be Filled By O.E.M.)
 deviceName = *********
 deviceType = Desktop
 deviceUniqueIdentifier = 996232ceb0e4469e709d9961100760ef0e6e4094
 graphicsDeviceID = 29679
 graphicsDeviceName = AMD Radeon RX 6650 XT
 graphicsDeviceType = Direct3D11
 graphicsDeviceVendor = ATI
 graphicsDeviceVendorID = 4098
 graphicsDeviceVersion = Direct3D 11.0 [level 11.1]
 graphicsMemorySize = 8147
 graphicsShaderLevel = 50
 npotSupport = Full
 maxTextureSize = 16384
 operatingSystem = Windows 10  (10.0.19045) 64bit
 processorCount = 12
 processorFrequency = 3693
 processorType = AMD Ryzen 5 5600X 6-Core Processor
 supportedRenderTargetCount = 8
 supports3DTextures = True
 supportsAccelerometer = False
 supportsComputeShaders = True
 supportsGyroscope = False
 supportsImageEffects = True
 supportsInstancing = True
 supportsLocationService = False
 supportsRawShadowDepthSampling = True
 supportsRenderTextures = True
 supportsRenderToCubemap = True
 supportsShadows = True
 supportsSparseTextures = True
 supportsStencil = 1
 supportsVibration = False
 systemMemorySize = 32693

Just to be sure. You put the launch option in the Subnautica Game and not in SteamVR?
I am out of ideas. I would maybe try reinstalling the game and see if you can run the game in SteamVR without the mod first.

The mod only works if it's running the SteamVR runtime. Removing the check would break it even more I am afraid. You can try building the mod without it but it's there for a reason.
You could try another streamer for the quest like ALVR or Virtual Desktop which always uses SteamVR but I am not sure why it's not working for you.

Oh, I heard steam link is pretty good too and super easy to setup.

i have ALVR. lemme try it

Still doesn't function with ALVR

What does it do? If it doesn't start in VR at all your SteamVR setup seems to be broken.

It starts in VR, the main menu loads and the headset works but the controllers don't, and the time i managed to click the required buttons to start a new save from the main menu, the controllers still aren't detected in-game

it's exactly the same as with Qlink

And you get the same log line with the game not being started in VR? The only way it could work is with SteamVR with alvr, so I am confused. I can build you a version without the check tonight but I am not sure it's gonna help.

Yep, same log line. The check-less version would be really helpful :)

Thank you! Unfortunately it still doesn't work, here's the error:
[Info   :   BepInEx] Loading [SubmersedVR 0.1.7]
[Error  :  HarmonyX] Failed to patch static bool GameInput::GetButtonDown(GameInput+Button action): System.Exception: Parameter "button" not found in method static bool GameInput::GetButtonDown(GameInput+Button action)
  at HarmonyLib.Public.Patching.HarmonyManipulator.EmitCallParameter (System.Reflection.MethodInfo patch, System.Boolean allowFirsParamPassthrough, Mono.Cecil.Cil.VariableDefinition& tmpObjectVar, System.Collections.Generic.List`1[HarmonyLib.Public.Patching.HarmonyManipulator+ArgumentBoxInfo]& tmpBoxVars) [0x007ae] in <474744d65d8e460fa08cd5fd82b5d65f>:0
  at HarmonyLib.Public.Patching.HarmonyManipulator.WritePrefixes (HarmonyLib.Internal.Util.ILEmitter+Label returnLabel) [0x00179] in <474744d65d8e460fa08cd5fd82b5d65f>:0
  at HarmonyLib.Public.Patching.HarmonyManipulator.WriteImpl () [0x0022c] in <474744d65d8e460fa08cd5fd82b5d65f>:0
[Error  : Unity Log] Exception: Parameter "button" not found in method static bool GameInput::GetButtonDown(GameInput+Button action)
Stack trace:
HarmonyLib.Public.Patching.HarmonyManipulator.EmitCallParameter (System.Reflection.MethodInfo patch, System.Boolean allowFirsParamPassthrough, Mono.Cecil.Cil.VariableDefinition& tmpObjectVar, System.Collections.Generic.List`1[HarmonyLib.Public.Patching.HarmonyManipulator+ArgumentBoxInfo]& tmpBoxVars) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Public.Patching.HarmonyManipulator.WritePrefixes (HarmonyLib.Internal.Util.ILEmitter+Label returnLabel) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Public.Patching.HarmonyManipulator.WriteImpl () (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
Rethrow as HarmonyException: IL Compile Error (unknown location)
HarmonyLib.Public.Patching.HarmonyManipulator.WriteImpl () (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Public.Patching.HarmonyManipulator.Process (MonoMod.Cil.ILContext ilContext, System.Reflection.MethodBase originalMethod) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Public.Patching.HarmonyManipulator.Manipulate (System.Reflection.MethodBase original, HarmonyLib.PatchInfo patchInfo, MonoMod.Cil.ILContext ctx) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Public.Patching.HarmonyManipulator.Manipulate (System.Reflection.MethodBase original, MonoMod.Cil.ILContext ctx) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Public.Patching.ManagedMethodPatcher.Manipulator (MonoMod.Cil.ILContext ctx) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
MonoMod.Cil.ILContext.Invoke (MonoMod.Cil.ILContext+Manipulator manip) (at <6733e342b5b549bba815373898724469>:0)
MonoMod.RuntimeDetour.ILHook+Context.InvokeManipulator (Mono.Cecil.MethodDefinition def, MonoMod.Cil.ILContext+Manipulator cb) (at <4e2760c7517c4ea79c633d67e84b319f>:0)
(wrapper dynamic-method) MonoMod.RuntimeDetour.ILHook+Context.DMD<MonoMod.RuntimeDetour.ILHook+Context::Refresh>(MonoMod.RuntimeDetour.ILHook/Context)
(wrapper dynamic-method) MonoMod.Utils.DynamicMethodDefinition.Trampoline<MonoMod.RuntimeDetour.ILHook+Context::Refresh>?1201442816(object)
HarmonyLib.Internal.RuntimeFixes.StackTraceFixes.OnILChainRefresh (System.Object self) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
MonoMod.RuntimeDetour.ILHook.Apply () (at <4e2760c7517c4ea79c633d67e84b319f>:0)
HarmonyLib.Public.Patching.ManagedMethodPatcher.DetourTo (System.Reflection.MethodBase replacement) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
Rethrow as HarmonyException: IL Compile Error (unknown location)
HarmonyLib.Public.Patching.ManagedMethodPatcher.DetourTo (System.Reflection.MethodBase replacement) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.PatchFunctions.UpdateWrapper (System.Reflection.MethodBase original, HarmonyLib.PatchInfo patchInfo) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
Rethrow as HarmonyException: IL Compile Error (unknown location)
HarmonyLib.PatchClassProcessor.ReportException (System.Exception exception, System.Reflection.MethodBase original) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.PatchClassProcessor.Patch () (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Harmony.<PatchAll>b__11_0 (System.Type type) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.CollectionExtensions.Do[T] (System.Collections.Generic.IEnumerable`1[T] sequence, System.Action`1[T] action) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Harmony.PatchAll (System.Reflection.Assembly assembly) (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
HarmonyLib.Harmony.PatchAll () (at <474744d65d8e460fa08cd5fd82b5d65f>:0)
SubmersedVR.Mod.Awake () (at <02d376fc0f4f4345878f2c6b6e3d3a27>:0)
UnityEngine.GameObject:AddComponent(Type)
BepInEx.Bootstrap.Chainloader:Start()
UnityEngine.InputSystem.InputSystem:.cctor()

Yeah I am afraid it would break. Are you using some old version of Subnautica? Only most recent version will work.

v72550

@Okabintaro issue is completely resolved, i updated the game to the march 2023 version and it works flawlessly, thanks!!!


next-preact - add preact to commons bundle

Fixes #24

This won't work with webpack 4. But it's also not required anymore.


Keychain cannot be found

When logging in to a mac that hasnt been logged in to before by that user. I get an error. A keychain cannot be found to store USERNAME@DOMAIN.
I am using the following prefrences
Create keychain = true
Add nomad to keychain = true
Reset keychain if password doesnt match = true
If i remove add nomad to keychain i can login but have no keychain
If i then run
Chmod 777 /Users/USERNAME/Library/Keychains
And enable add nomad to keychain
Then logout and backin. A keychain is created.
The premissions on the keychain folder before was USERNAME:staff rwx--x--x or 711
Tested on 12.2 1 and 11.6.4
If i manually create the folder it works
Sudo mkdir /Users/USERNAME
sudo chown USERNAME /Users/USERNAME

I have a shared lab using Nomad where this just started happening as well, seemingly at random. But it doesn't seem to be happening at other buildings with the exact same setup. Did you ever find out what this is? There's not much official Nomad support at this point.

No i gave up couldnt get it to work reliably shame as when it works its good

From: Brad White @.>
Sent: 19 April 2022 20:32
To: jamf/NoMADLogin-AD @.>
Cc: chrism9876 @.>; Author @.>
Subject: Re: [jamf/NoMADLogin-AD] Keychain cannot be found (Issue #2)
I have a shared lab using Nomad where this just started happening as well, seemingly at random. But it doesn't seem to be happening at other buildings with the exact same setup. Did you ever find out what this is? There's not much official Nomad support at this point.
—
Reply to this email directly, view it on GitHubhttps://github.com/jamf/NoMADLogin-AD/issues/2#issuecomment-1103017847, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF2PKD75EP5FAF5TONX2W7TVF4C57ANCNFSM5Q4JHIUA.
You are receiving this because you authored the thread.Message ID: @.***>

That's a shame. We're getting the same error, but it seems our issue is the opposite - new people are logging in fine, it's users with existing accounts that have this problem happen. Supposedly upgrading one of the computers to Monterey fixed it, but can't 100% confirm. Again, just strange as we have multiple other labs with identical setups, but this is the only one where it's happening.


Large size of application.js produced by esbuild

Hello, everyone! This issue is about finding a solution for a large-size application.js file produced by the jsbundling-rails+esbuild+sprokets bundle.
In this file, we provide esbuild with an --bundle option, which leads each file compiled to app/assets/builds to have included all imported modules
https://github.com/rails/jsbundling-rails/blob/c89eb022c58475999b845c67fffb42b62f57ee28/lib/install/esbuild/install.rb#L5
After the assets pipeline processes these files, you get an output application.js file with many libraries imported multiple times. In my case, there are ~100 inclusions of React and other libraries, and the raw produced file has a 95MB size. I can make it look better by adding the --minify option, which will reduce the size of the application.js to 27MB, but the problem with repeated imports remains the same.
There is a discussion of this problem in https://github.com/evanw/esbuild/issues/475, and it looks like the author of esbuild does not take clever linking as a priority (https://github.com/evanw/esbuild/issues/475#issuecomment-714358171 with https://github.com/evanw/esbuild/issues/475#issuecomment-1156911260 and https://github.com/evanw/esbuild/issues/2818#issuecomment-1381040064).
Would you happen to have any suggestions on how to approach this issue? My only thought now is switching to webpack (probably with esbuild-loader).

I resolved this issue by adding a main.js file that imported all other files and slightly changing the configuration for esbuild in esbuild.config.js like this:
build({
  entryPoints: [path.join(process.cwd(), "app/javascript/main.js")],
  bundle: true,
  minify: true,
  format: "iife",
  allowOverwrite: true,
  outdir: path.join(process.cwd(), "app/assets/builds"),
  absWorkingDir: path.join(process.cwd(), "app/javascript"),
  sourcemap: true,
  plugins: [ImportGlobPlugin.default()],
  target: browserslistToEsbuild(targets)
})


Add tests for specific HTTP libraries & version

We see a number of issues around nock not working with specific node HTTP libraries (ex #748).
We should create subdirectories within tests that include their own package.json and test suite, so we may test nock against different HTTP libraries and versions thereof.
Please use the Newrelic tests as a model for what this would look like https://github.com/newrelic/node-newrelic/tree/master/test/versioned

Done


How to compile and run KITE test in Linux ?

Im about to start writing KITE tests in Linux. This is the Dockerfile i created:
FROM ubuntu as KiteBase
RUN apt update
RUN apt install -y default-jre default-jdk openjdk-8-jdk maven git gcc build-essential
RUN git clone https://github.com/webrtc/KITE.git KITE
WORKDIR /KITE/scripts/linux
RUN chmod +x *.sh ./setupLocalGrid.sh
COPY ./utils/google-chrome.list /etc/apt/sources.list.d/google-chrome.list
RUN apt install wget -y
RUN wget https://dl.google.com/linux/linux_signing_key.pub
RUN apt-key add linux_signing_key.pub
RUN apt update
RUN apt install google-chrome-stable -y
RUN apt install npm  nodejs -y
RUN nodejs -v
ENV KITE_HOME=/KITE
ENV MAVEN_HOME =/usr/share/maven
ENV JAVA_HOME =/usr/share/Java
FROM KiteBase
COPY ./scripts/linux/setupLocalGrid.sh .
COPY ./KITE-Example-Test .
I am able to create the localgrid in linux like this:

Im able to go into the project folder like in the picture below :

Now I'm asking if i should at first compile the project and then run it ?! If yes, how can I do the compiling and running the project, because the commands provided in this tutorial are not working :

Thanks!

Hi,
After cloning KITE, you can run our configuring scripts (depends on your OS):

configureLinux.sh
configure.bat
configureMac.sh

This will set the env variables for KITE, including the path to the scripts (KITE/scripts/os/path/) for:

c : compiling
r : run
a : open allure report (kite-allure-reports)

After this you can use the commands in the tutorials. We do recommend you running the configure script, but alternatively, you can use these followings:

Compile: mvn -DskiptTests clean install
Run: java (-Dkite.firefox.profile="PATH_TO_FIREFOX_PROFILE) -cp PATH_TO_KITE/KITE-Engine/target/kite-jar-with-dependencies.jar:PATH_TO_TEST/target:*" org.webrtc.Engine PATH_TO_CONFIG_FILE
Allure report: allure serve PATH_TO_ALLURE_REPORT

*Firefox profile is optional, however, we do provide a default profile with h264 plugin pre-installed, this might be needed for some WebRTC application.
**For classpath, on Linux and Mac, the jars are separated by ":", and on Windows by ";"
Cheers,

I did all the instructions you said and i copied the project , and now I want to run the KITE-Example test, but im stucked with this error :

Did you compile the KITE Engine before cd into KITE-Example-Test and call c?
If you already call c int KITE/, it compiles all of the modules including KITE-Example-Test, there's no need to compile the test module individually unless you're making changes to it.
After that you can just call r to run your test.

When im trying to compile the entire KITE folder , it is showing this :

I'm trying to reproduce your errors, but can't. This is on a virtual machine with Ubuntu 18.04, maybe you're using a different variation of Linux?

Java and Maven version:

This is the docker file im working with :
FROM ubuntu as KiteBase
RUN apt update
RUN apt install -y default-jre default-jdk openjdk-8-jdk maven git gcc build-essential nodejs npm
RUN apt-get install -y wget
RUN wget https://dl.google.com/linux/linux_signing_key.pub
#KITE
RUN git clone https://github.com/webrtc/KITE.git KITE
#RUN mvn --version
#RUN java --version
#WORKDIR /KITE/
#RUN chmod +x *.sh ./configureLinux.sh
WORKDIR /KITE/scripts/linux
RUN chmod +x *.sh ./setupLocalGrid.sh
COPY ./utils/google-chrome.list /etc/apt/sources.list.d/google-chrome.list
RUN apt install wget -y
RUN wget https://dl.google.com/linux/linux_signing_key.pub
RUN apt-key add linux_signing_key.pub
RUN apt update
RUN apt install google-chrome-stable -y
#RUN apt install npm  nodejs -y
ENV KITE_HOME=/KITE
ENV JAVA_HOME =/usr/lib/jvm/java-11-openjdk-amd64
RUN export JAVA_HOME
ENV MAVEN_HOME =  /usr/share/maven
RUN export MAVEN_HOME
FROM KiteBase
COPY ./scripts/linux/setupLocalGrid.sh .
COPY ./scripts/linux/gridConfig.sh .
COPY ./scripts/linux/path/* ./
#COPY ./scripts/linux/path/c .
#COPY ./scripts/linux/path/r .
#COPY ./scripts/linux/path/t .
#COPY ./scripts/linux/path/a .
#COPY ./scripts/linux/path/kite_init .
COPY ./KITE-Example-Test/* ./
Im still able to launch the grid :

While, when i try to compile KITE or any project or check the maven version im getting this error :

I can't really help if I can't reproduce the issue here.
I followed your set of commands in both of your docker files and I got:

Java 11, maven 3.6 (my previous comment)
Java 8, maven 3.3.9 :

For me, on Ubuntu 18.04, these commands should be enough to compile KITE:
sudo apt-get install git openjdk-8-jdk maven
git clone https://github.com/webrtc/KITE KITE
cd KITE
cp scripts/linux/path/* ./ (if you don't want to add this to you $PATH)
./c

Which variation of LINUX are you using?

But probably it should be an alternative to run tests or to recognize the 'r' command :

FROM ubuntu as KiteBase
RUN apt update
RUN apt install -y default-jre default-jdk openjdk-8-jdk maven git gcc build-essential nodejs npm
RUN apt-get install -y wget
RUN wget https://dl.google.com/linux/linux_signing_key.pub
#KITE
RUN git clone https://github.com/webrtc/KITE.git KITE
RUN mvn --version
RUN java --version
#WORKDIR /KITE/
#RUN chmod +x *.sh ./configureLinux.sh
WORKDIR /KITE/scripts/linux
RUN chmod +x *.sh ./setupLocalGrid.sh
COPY ./utils/google-chrome.list /etc/apt/sources.list.d/google-chrome.list
RUN apt install wget -y
RUN wget https://dl.google.com/linux/linux_signing_key.pub
RUN apt-key add linux_signing_key.pub
RUN apt update
RUN apt install google-chrome-stable -y
ENV KITE_HOME=/KITE
ENV JAVA_HOME /usr/lib/jvm/java-11-openjdk-amd64
RUN export JAVA_HOME=/opt/java/jdk-11.0.1
RUN export PATH=$PATH:$JAVA_HOME/bin
ENV MAVEN_HOME  /opt/maven
RUN export MAVEN_HOME
FROM KiteBase
COPY ./scripts/linux/setupLocalGrid.sh .
COPY ./scripts/linux/gridConfig.sh .
COPY ./scripts/linux/path/c .
COPY ./scripts/linux/path/r .
COPY ./scripts/linux/path/t .
COPY ./scripts/linux/path/a .
COPY ./scripts/linux/path/kite_init .
#WORKDIR /KITE-Example-Test .
#COPY ./KITE-Example-Test .

There is of course, the r command is just a quick way to do it instead of:
java (-Dkite.firefox.profile="PATH_TO_FIREFOX_PROFILE") (-Dkite.chrome.extension="PATH_TO_CHROME_EXTENSION")  -cp "PATH_TO_kite-jar-with-dependencies.jar:PATH_TO_YOUR_TEST_JAR" org.webrtc.kite.Engine PATH_TO_YOUR_CONFIG_FILE

any (...) is optional
kite-jar-with-dependencies.jar is normally at KITE/KITE-Engine/target after compiling
your test jar is normally in the target folder of your test folder
on windows, to separate classpath, it's ";" instead of ":"

@KiteTesting can you share your actual docker commands (docker build etc.)

FWIW I was able to build the image with your Dockerfile with this modification:
FROM ubuntu:bionic-20191029 as KiteBase

and
KITE$ cat utils/google-chrome.list
### THIS FILE IS AUTOMATICALLY CONFIGURED ###
# You may comment out this entry, but any other modifications may be lost.
deb [arch=amd64] http://dl.google.com/linux/chrome/deb/ stable main

I've been able at first to launch the grid, then to compile and run the project in nodes

But I keep getting those erros about webdriver, any help about that ?
This is the dockerfile im using :
FROM ubuntu:bionic-20191029 as KiteBase
RUN apt update
RUN apt install -y default-jre default-jdk openjdk-8-jdk maven git gcc build-essential nodejs npm
RUN apt-get install nano -y
RUN apt-get install -y wget
RUN wget https://dl.google.com/linux/linux_signing_key.pub
#KITE
RUN git clone https://github.com/webrtc/KITE.git KITE
RUN mvn --version
RUN java --version
WORKDIR /KITE/
WORKDIR /KITE/
RUN chmod +x *.sh ./configureLinux.sh
COPY ./utils/google-chrome.list /etc/apt/sources.list.d/google-chrome.list
RUN wget https://dl.google.com/linux/linux_signing_key.pub
RUN apt-key add linux_signing_key.pub
RUN apt update
RUN apt install google-chrome-stable -y
ENV KITE_HOME=/KITE
RUN export KITE_HOME
ENV JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64
RUN export JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64/bin
#RUN export PATH=$PATH:$JAVA_HOME/bin
ENV MAVEN_HOME=/opt/maven
RUN export MAVEN_HOME
FROM KiteBase
COPY ./KITE-Engine .
COPY ./KITE-Framework .
COPY ./KITE-Engine .
COPY ./scripts/linux/setupLocalGrid.sh .
COPY ./scripts/linux/gridConfig.sh .
COPY ./scripts/linux/path/c .
COPY ./scripts/linux/path/r .
COPY ./scripts/linux/path/t .
COPY ./scripts/linux/path/a .
COPY ./scripts/linux/path/kite_init .
ADD pom.xml /KITE/pom.xml
#WORKDIR ./KITE-Engine .
#COPY ./KITE-Engine .
WORKDIR KITE/KITE-MessengerLogin-Test .
COPY ./KITE-MessengerLogin-Test .

Hi, I'll try to reproduce that error with Js test next week and get back to you. Normally if your grid has chrome on linux, there should not be this error.
However we are not currently using docker, so I will test it only with ubuntu 18.

Now, with this version of dockerfile, im getting this error:
FROM ubuntu as KiteBase
RUN apt update
RUN apt install -y default-jre default-jdk openjdk-8-jdk maven git gcc build-essential nodejs npm
RUN apt-get install nano -y
RUN apt-get install -y wget dpkg unzip
#KITE
RUN git clone https://github.com/webrtc/KITE.git KITE
RUN mvn --version
RUN java --version
WORKDIR /KITE/
RUN chmod +x *.sh ./configureLinux.sh
ENV KITE_HOME=/KITE
RUN export KITE_HOME
ENV JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64
RUN export JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64/bin
#RUN export PATH=$PATH:$JAVA_HOME/bin
ENV MAVEN_HOME=/usr/share/maven
RUN export MAVEN_HOME
FROM KiteBase
COPY ./KITE-Engine .
COPY ./KITE-Framework .
COPY ./KITE-Engine .
COPY ./scripts/linux/setupLocalGrid.sh .
COPY ./scripts/linux/gridConfig.sh .
COPY ./scripts/linux/path/c .
COPY ./scripts/linux/path/r .
COPY ./scripts/linux/path/t .
COPY ./scripts/linux/path/a .
COPY ./scripts/linux/path/kite_init .
COPY ./scripts/linux/createFolderLocalGrid.sh .
COPY ./scripts/linux/installChrome.sh .
COPY ./scripts/linux/installFirefox.sh .
COPY ./scripts/linux/installSelenium.sh .
COPY ./scripts/linux/installMaven.sh .
COPY ./scripts/linux/interactiveInstallation.sh .
ADD pom.xml /KITE/pom.xml
WORKDIR KITE/KITE-MessengerLogin-Test .
COPY ./KITE-MessengerLogin-Test .
Check the errors here https://prnt.sc/qg0j6n https://prnt.sc/qg0jfj https://prnt.sc/qg0kd9
I know the errors are about the drivers versions, but could you help us a little here ?

It looks like the error is due to the connection to the hub, the localhost  address is not reachable for KITE.

FROM ubuntu as KiteBase
RUN apt update
RUN apt install -y default-jre default-jdk openjdk-8-jdk maven git gcc build-essential
RUN apt-get install nano -y
RUN apt-get install -y sudo wget dpkg unzip xvfb
#KITE
RUN git clone https://github.com/webrtc/KITE.git KITE
RUN mvn --version
RUN java --version
WORKDIR /KITE/
RUN chmod +x *.sh ./configureLinux.sh
ENV KITE_HOME=/KITE
RUN export KITE_HOME
ENV JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64
RUN export JAVA_HOME=/usr/lib/jvm/java-8-openjdk-amd64/bin
#RUN export PATH=$PATH:$JAVA_HOME/bin
ENV MAVEN_HOME=/usr/share/maven
RUN export MAVEN_HOME
FROM KiteBase
COPY ./KITE-Engine .
COPY ./KITE-Framework .
COPY ./KITE-Engine .
COPY ./scripts/linux/setupLocalGrid.sh .
COPY ./scripts/linux/gridConfig.sh .
COPY ./scripts/linux/path/c .
COPY ./scripts/linux/path/r .
COPY ./scripts/linux/path/t .
COPY ./scripts/linux/path/a .
COPY ./scripts/linux/path/kite_init .
COPY ./scripts/linux/createFolderLocalGrid.sh .
COPY ./scripts/linux/installChrome.sh .
COPY ./scripts/linux/installFirefox.sh .
COPY ./scripts/linux/installSelenium.sh .
COPY ./scripts/linux/installMaven.sh .
COPY ./scripts/linux/installDrivers.sh .
COPY ./scripts/linux/interactiveInstallation.sh .
ADD pom.xml /KITE/pom.xml
WORKDIR KITE/KITE-Login-Test .
COPY ./KITE-Login-Test .
From this dockerfile i launch the grid by this command RUN chmod +x *.sh ./configureLinux.sh .
the script installChrome.sh::
#!/bin/bash
set +v
. ./gridConfig.sh
google-chrome-stable -version >chrome.txt
chromeFile="chrome.txt"
chrome=$(cat "$chromeFile")
if [[ $chrome == $CHROME_VERSION ]]; then
echo Chrome version $CHROME_VERSION was found
else
echo Chrome version $CHROME_VERSION was not found, installing it.
wget https://dl.google.com/linux/direct/google-chrome-stable_current_amd64.deb
dpkg -i google-chrome-stable_current_amd64.deb
apt-get fonts-liberation
apt --fix-broken install
rm -f google-chrome-stable_current_amd64.deb
fi
rm -f chrome.txt
exit
the script setupLocalGrid.sh :
#!/bin/bash
set +v
. ./gridConfig.sh
echo $KITE_HOME
cd $KITE_HOME/scripts/linux
function startGrid() {
cd $KITE_HOME/localGrid
if [ "$DESKTOP_ENVIRONMENT" = "TRUE" ]
then
x-terminal-emulator -e ./startGrid.sh &
else
./startGrid.sh &
fi
}
if [[ "$DESKTOP_ENVIRONMENT" = "TRUE" ]]
then
apt-get -y upgrade -f gnome-terminal
fi
./createFolderLocalGrid.sh
if [[ "$INSTALL_BROWSERS" = "TRUE" ]]
then
./installChrome.sh
./installFirefox.sh
else
echo "Skipping Chrome and Firefox installation"
fi
./installDrivers.sh
./installSelenium.sh
#read -p "Do you want to start the Grid (y/n)?  " yn
#case $yn in
startGrid
;;
# [Nn]* )
 #      echo " "
  #     ;;
 #* ) echo "Please answer yes or no.";;

#esac
#startGrid
echo "Setup completed."
gridConfig.sh script:
#!/bin/bash
Grid setup configuration for Ubuntu

Please check the Firefox and Chrome versions installed on your computer. If none installed, this script will download and install them.

Please check and update the corresponding chromedriver and geckodriver versions from:
http://chromedriver.chromium.org/downloads
https://github.com/mozilla/geckodriver/releases

Please check the selenium version from: https://selenium-release.storage.googleapis.com/
Set to TRUE to install Chrome and Firefox, or FALSE to skip this step if the correct versions are already installed.
export INSTALL_BROWSERS=TRUE
Set to TRUE/FALSE to enable/disable the CAPABILITY MATCHER
export USE_CAPABILITY_MATCHER=FALSE
Set to TRUE if this is a full desktop environment, FALSE if it's without any display
export DESKTOP_ENVIRONMENT=FALSE
Set to TRUE to use localhost or FALSE to use the host's IP address
export LOCALHOST=FALSE
ChromeDriver and GeckoDriver versions
export GECKO_VERSION=v0.26.0
export CHROMEDRIVER_VERSION=79.0.3945.36
Selenium version
export SELENIUM_VERSION_SHORT=3.141
export SELENIUM_VERSION=3.141.59
Browser versions
export FIREFOX_VERSION=70
export CHROME_VERSION=79
#Capability Matcher
export KITE_EXTRAS_VERSION=0.1.4
export GRID_UTILS_VERSION=0.0.1

And on the grid's console UI (http://HUB_IP:HUB_PORT/grid/console), do you have all the browsers listed, with the correct config that you're calling ?
Something like this: (never mind the gateway and binary path)

and same for firefox

Im testing only with Chrome and this is what i get :

Looks like you do have a node registered with chrome 79.
Does you config file look like this, for the grid and clients?

Can the machine/container on which you're running KITE connect to http://<IP_ADDRESS>:4444/wd/hub
Please provide the exact info that I require and make sure that your comments are easy to read.
It seems that every time you are providing a lot of  irrelevant stuff which make it difficult to understand your process and to resolve your issue.

issue is solved

@KiteTesting what was your eventual Dockerfile?  I have built one myself, but I can't seem to get Chrome to run headless.


mps-youtube: 0.2.8 -> unstable-2010-01-28

Motivation for this change
mps-youtube-0.2.8 is no longer compatible with the version of mpv included in nixpkgs (cf this issue).
There haven't been any mps-youtube release since then, so the only way to fix it is to point at a git revision.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Works on my side:
$ grep completer /nix/store/d3yrji9kp1kn2jvnb056wlkz60k8503d-python3.7-mps-youtube-unstable-2020-01-28                                                                                Sun 16 Feb 2020 06:52:34 PM CET
/nix/store/d3yrji9kp1kn2jvnb056wlkz60k8503d-python3.7-mps-youtube-unstable-2020-01-28/lib/python3.7/site-packages/mps_youtube/main.py
34:completer = None
39:    completer = util.CommandCompleter()
41:    readline.set_completer(completer.complete_command)
42:    readline.set_completer_delims('')

/nix/store/d3yrji9kp1kn2jvnb056wlkz60k8503d-python3.7-mps-youtube-unstable-2020-01-28/lib/python3.7/site-packages/mps_youtube/commands/__init__.py
5:from ..main import completer
28:        completer.add_cmd(command)

completer is well defined inside main module.

is this supposed to be used as a standalone application? (like a cli utility?)

Yes, this is a CLI application.

The reason why I ask is that we could export this as a top level mps-youtube command using buildPythonApplication, and not mess with package (which is much more fragile as to what environment is calling it)

I've just replaced buildPythonPackage with buildPythonApplication, as suggested. Today I learned :) .

Moved to pkgs/applications/misc, added deprecation notice in python-packages.

would still like @FRidh to sign off
@worldofpeace do you have any strong feelings?

LGTM, for what it's worth

@k0ral Your commit message has the wrong date https://github.com/NixOS/nixpkgs/pull/80269/commits/785fe193b6d12aa306d31317442dc30a821c5e82.
It should be 2020-01-28 not 2010

Updated with all your comments taken into account.

Squashed commits.


add standard upgrade support from extended

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/eks_cluster#upgrade_policy
https://github.com/hashicorp/terraform-provider-aws/issues/38510
in provider
https://github.com/hashicorp/terraform-provider-aws/blob/v5.61.0/CHANGELOG.md
https://developer.hashicorp.com/terraform/language/providers/configuration#provider-versions

michaelobrien@mbp7 modules % terraform plan
╷
│ Error: Inconsistent dependency lock file
│
│ The following dependency selections recorded in the lock file are inconsistent with the current configuration:
│   - provider registry.terraform.io/hashicorp/aws: locked version selection 4.67.0 doesn't match the updated version constraints "~> 5.61"
│
│ To update the locked dependency selections to match a changed configuration, run:
│   terraform init -upgrade
╵
michaelobrien@mbp7 modules % terraform init -upgrade
Initializing the backend...
Initializing provider plugins...
- Finding hashicorp/aws versions matching "~> 5.61"...
- Installing hashicorp/aws v5.73.0...
- Installed hashicorp/aws v5.73.0 (signed by HashiCorp)

https://docs.aws.amazon.com/eks/latest/userguide/disable-extended-support.html
michaelobrien@mbp7 modules % aws eks update-cluster-config --name example-eks-cluster --upgrade-policy supportType=STANDARD --region us-east-1
{
    "update": {
        "id": "7ed67c07-a623-3279-966a-3da15f4e02fa",
        "status": "Successful",
        "type": "UpgradePolicyUpdate",
        "params": [
            {
                "type": "UpgradePolicy",
                "value": "{\"supportType\":\"STANDARD\"}"
            }
        ],
        "createdAt": "2024-10-30T12:11:09.340000-04:00",
        "errors": []
    }
}

michaelobrien@mbp7 modules % aws eks describe-cluster --name example-eks-cluster --query "cluster.upgradePolicy.supportType" --region us-east-1
"STANDARD"


hostapd: 2.8 -> 2.9

Associated release note:
https://w1.fi/security/2019-6/sae-eap-pwd-side-channel-attack-update.txt
Motivation for this change
Our current hostapd setup might be vulnerable to a side-channel attack on EAP-pwd in a particular setup.
Relevant release note part:

All wpa_supplicant and hostapd versions with EAP-pwd support
(CONFIG_EAP_PWD=y in the build configuration and EAP-pwd being enabled
in the runtime configuration). Note that EAP-pwd server implementation
in hostapd enables only a single group at the time (pwd_group parameter)
and by default, group 19 is used. As such, this would be applicable only
if the pwd_group parameter is set to use one of the groups 28-30. The
EAP-pwd peer implementation wpa_supplicant, follows the group selected
by the server and as such, it would be vulnerable for the case where an
attacker controls the authentication server (e.g., through a rogue AP)
if the crypto library supports groups 28-30.

No reason to freak about that, but better be safe than sorry.
There's no breaking change regarding this release, updating hostapd should be safe and backward-compatible.
We probably want to backport this to 19.09 together with https://github.com/NixOS/nixpkgs/pull/75140 (not merged yet).
Tested on my test router setup.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @flokli

Thanks @teto !

backported to 19.09 in c7f71b572b9.


TTS overrides previous users' message if another request is done right after the first one

Is it possible to add a cooldown timer so that the next request is only processed after a certain timer value set to prevent the requests from overriding/overlapping?

Done by @Vuurvos1 in cdd1cdffc15bcd245d4b22e7977415cad2b1bb7c !


:sparkles: Add Added a simple test.

Issue

close #300

Overview (Required)

Test that the title of the notification screen is displayed.

I thought it would be better to add a test for each of JP and EN, but I couldn't figure out how to test for each language. 🤔

Links

Nothing.

👀


feat: add phalcon 5.8.0 release

update phalcon with release 5.8.0

:tada: This issue has been resolved in version 1.323.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Columns with default values breaks validation and causes queries to return invalid data

Describe the bug
When a model contains a column with a default value this causes queries to run into validation errors. I believe this is because ormar does not fetch the whole state of related models but rather only returns the primary key unless requested explicitly with .select_related(). A model with defaults breaks this behavior and instead returns the primary key and populates default values, excluding other required fields.
To Reproduce
from typing import Optional
from fastapi import FastAPI

import databases
import pydantic

import ormar
import sqlalchemy

DATABASE_URL = "sqlite:///bug.sqlite"
database = databases.Database(DATABASE_URL)
metadata = sqlalchemy.MetaData()

class BaseMeta(ormar.ModelMeta):
    metadata = metadata
    database = database

class Author(ormar.Model):
    class Meta(BaseMeta):
        tablename = "authors"

    id: int = ormar.Integer(primary_key=True)
    name: str = ormar.String(max_length=100)
    rating: int = ormar.Integer(default=0) #<------------- default value

class Book(ormar.Model):
    class Meta(BaseMeta):
        tablename = "books"

    id: int = ormar.Integer(primary_key=True)
    author: Optional[Author] = ormar.ForeignKey(Author)
    title: str = ormar.String(max_length=100)
    year: int = ormar.Integer(nullable=True)

engine = sqlalchemy.create_engine(DATABASE_URL)
# just to be sure we clear the db before
metadata.drop_all(engine)
metadata.create_all(engine)

author = Author(name='bug', rating=5)
book = Book(author=author, title='Bug caused by default value', year=2021)

app = FastAPI()

@app.on_event('startup')
async def startup():
    await author.save()
    await book.save()

@app.get('/{id}', response_model=Book)
async def get_book_by_id(id: int):
    book = await Book.objects.get(id=id)
    print(book)
    return book

Running the above code, then requesting the book we created causes a validation error:
pydantic.error_wrappers.ValidationError: 1 validation error for Book
response -> name
  field required (type=value_error.missing)

The print out of the book object before returning the response is:
id=1 author=Author({'id': 1, 'name': None, 'rating': 0, 'books': [<weakproxy at 0x7f476effaa40 to Book at 0x7f47700a3b20>]}) title='Bug caused by default value' year=2021

As you can see here, the instance of Author in the book we retrieved has invalid state. Its 'name' value is None and has the default rating value populated, but that violates the pydantic models validation which occurs when FastAPI constructs the response.
Expected behavior
I expect ormar to not return these default values for related models. This is extra dangerous because you could mistakenly think that the default values returned reflect the actual state of the object.
Versions

Database backend used sqlite
Python version 3.9.4
ormar version 0.10.5
pydantic version 1.8
fastapi version 0.63.0

Yeah that should be a pk only model with just pk populated, something is wrong here, will check that.
Thanks for catching that 🙂

Fixed in 0.10.8 (#203)

Wow, quick fix! Thanks for the useful python package.


Add documentation demonstrating use as ring signature scheme, both linkable and unlinkable variants.

A ring signature scheme may be constructed, by committing to some message text in the proof transcript before proving/verifying each proof. Optionally, the use of offsets may lead to a linkable ring signature, to identify multiple signatures from the same member (e.g. linking signatures).

Resolved in https://github.com/cargodog/one-of-many-proofs/pull/26


add test url

seems testUrl in newer jest version: https://github.com/facebook/jest/issues/6766 https://github.com/jsdom/jsdom/issues/2304

ping @rmariuzzo


Minor fixes to support iOS targets (with packages that depend on it ex. MongoKitten & Meow).

Hey there,
These modifications are necessary to allow packages using MongoKitten + Meow to target iOS which otherwise would result in a failing build.
The modifications needed are as follows:
1.) Useless variable, which is untyped, a build error.
https://github.com/Wabi-Studios/NioDNS/commit/f8c52ff6287b2935671bb0bff79429940ac6f70d
2.) Should NIOTransportServices be added as a dependency?
https://github.com/Wabi-Studios/NioDNS/commit/5e3b84b3c926e3b578befd69ceeffc7a50cd5fb3

Thanks! Fixed and tagged.


Inspector: Capture screenshot produces transparent image when engine preserveDrawingBuffer is false

Inspector's capture screenshot (See features https://doc.babylonjs.com/features/playground_debuglayer) produces transparent screenshot when Engine created with option preserveDrawingBuffer: false (default). There should be at least warning about it in the console if it is impossible.
Bugs

Bug repro on playground: Can not reproduce it in the playground - Engine is initiated with preserveDrawingBuffer: true. Download any playground, change preserveDrawingBuffer to false (or remove this option) and try to take a screenshot via the inspector.
Expected result: Correctly capture scene screenshot.
Current result: Captured screenshot is transparent.

Bugged screenshot - transparent:

Hey please use the forum to discuss. This is by default. We mentioned it in the doc: https://doc.babylonjs.com/how_to/render_scene_on_a_png#createscreenshot

I was searching for it through the inspector features - https://doc.babylonjs.com/features/playground_debuglayer. Maybe it is worth to add a note about it there too?

Sure (or at least point to the screenshot doc :))
Fancy doing a PR?


"Singular" rate ore blocks can't be melted

Minecraft Version
1.20.1
Fabric Version
0.14.23
Fabric API Version
0.89.3+1.20.1
Hephaestus Version
1.20.1-<IP_ADDRESS>
Describe your issue
Ore blocks that have a "singular" melting rate can't be molten at all.
This affect not only vanilla gameplay, but addons (which use the methods from ISmelteryRecipeHelper to automatically generate the needed .json files) and datapacks too.
Ores with "sparse" (like Nether Gold Ore) and "dense" (like Copper Ore) rates can be molten normally.
Screenshots
Iron Ore (singular)

(notice the lack of Melter tab)
Copper Ore (dense)

Nether Gold Ore (sparse)

Crash Report
No response
Other mods
A devenv, which also runs:

Mythic Metals 0.18.2+1.20
Alloy Forgery 2.0.21+1.20
OwOlib 0.11.1+1.20

Tried reproducing with just Tinkers?
No
Performance Enchancers
None of the above
Searched for known issues?
Searched open issues, Searched closed issues

Seems like it's fixed in v1.20.1-<IP_ADDRESS>.


Sceneform sdk should pass through Proguard rules from Filement artifact

https://github.com/ThomasGorisse/sceneform-android-sdk/issues/69#issuecomment-849302503
The workaround described in this should not be needed...Filament has proguard rules for UsedBy:
https://github.com/google/filament/blob/24b88219fa6148b8004f230b377f163e6f184d65/android/proguard-rules.pro
They should be passed through and included in this sdk's proguard rules.

I can confirm that these ProGuard rules need to be preserved in Sceneform. We keep them up to date so it's best to rely on us doing the right thing here.

I'm making a change so that Filament will properly include the ProGuard rules file in its AARs. You may need to add your own consumerProguardFiles directive in build.gradle to preserve our rules though.

Also related : google-ar/arcore-android-sdk/issues/1232
@kenyee Can you make a Pull Request with the current rules?
I will keep the PR branch alive so that we can keep in mind that there will have modifications on this Filament part.

@ThomasGorisse :
You mean add a temp one w/ a proguard file for Filament until Filament adds proguard rules properly?  Or do you mean fix how this repo handles it's rules?

I just made the change on our end, the next release of Filament (next week) should have it.

Do we need to do something to forward your rules ?
It's been a while since I worked with proguard but from what I remember we should put the view and fragment class cause they aren't marked as used from the layout res file?
I will double check but the goal for the next release will be to remove all the still living sfb classes and so (I hope) all the actual jni part from Sceneform so there might not need to be put in the proguard rules.

IIRC, as long as you include filament w/ "api" in your gradle file, the proguard file should be passed through.
Question is whether you have anything in this sceneform sdk that needs proguard exclusions too...then we'd need to add "consumerProguardFile" against this SDK's proguard rules file.

Couldn't find a proguard file in the sceneform SDK...should there be one?
Filament seems to be pulled in via an "api" gradle command so that should be fine.

I think it's better to wait for the SFB remove since there could have quite a lot of rules to clean just after trying to find them on a previous commit.


Remove unused jenkins scripts

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #4108 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4108   +/-   ##
=======================================
  Coverage   70.49%   70.49%
=======================================
  Files         437      437
  Lines       28134    28134
  Branches     4316     4316
=======================================
  Hits        19832    19832
- Misses       7014     7018    +4
+ Partials     1288     1284    -4

Impacted Files
Coverage Δ

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
70.66% <0%> (ø)
:arrow_up:

...li-cloud/azure/cli/command_modules/cloud/custom.py
13.79% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
: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 ee252bc...163b548. Read the comment docs.


Compiling on Linux environment

Can you please give some guide on how to work with your code on linux ubuntu environment.
Thanks for your code and for your help beforehand.

Can you please guid which file exactly to compile.
I am just litlle frusturates and do not know which to use and which to compile.


Add concatenate function

Which issue(s) are closed by this pull request?
Closes #
Changes proposed in this pull request:

Add a concatenation function, concatenating Signals along specified channels.
Add broadcasting parameter, which broadcasts the signals into largest cdim and a common shape, ignoring caxis.

I tried to adjust the comment of the merged signal so that the comments can be assigned to the data in the merged signal. First, to the comment of the merged signal is passed on which caxis the signals were merged. If there is a comment in one of the input Signals, that comment is appended, as well as the corresponding channels on caxis.
Questions:

Is it right to call it channels?
Do we start counting at 0 or at 1? For example first_signal.cshape[caxis] is 3, would we say the comment corresponds to channel 1-3 or 0-2?


Improve Reload

We were not able to use electron-reloader. Think about a better way to reload (currently it restarts the app on build)

Maybe something from https://github.com/sprinteins/ghost/pull/71/commits/44a9a991e01fcdbd4d4d56f28381319951cc3b1e reloader.js is reuseable


Rancher UI cannot support to adjust the column width

Rancher versions:
rancher/server:1.6.2
rancher/agent:1.2.2
Steps to Reproduce:
Open INFRASTRUCTURE -> HOSTS, select one hosts and open a new page.
Results:
The containers grid cannot adjust the column width, the container's name is hiding.
Please check the image below:

We're not going to do resizable columns, but they are better laid out / 2 rows and no left sidebar in 2.0 so there is room to see more of them.


Low performance for encoding/decoding when using custom member names transformation with semiauto

The issue is occured when member names should be rewritten, i.e. with using io.circe.generic.extras.Configuration.default.withSnakeCaseMemberNames regexp transformation are occured on every json encoding/decoding (see io.circe.generic.extras.ConfiguredDecoder, io.circe.generic.extras.ConfiguredObjectEncoder).
I think they should be performed only once when encoder/decoder are derived.

+1
I had the same issue. As a workaround I had to cache these member names:
private val fieldNamesCache = TrieMap[String, String]()

implicit val config: Configuration = Configuration.default.copy(transformMemberNames = str =>
  fieldNamesCache.getOrElseUpdate(str, Configuration.snakeCaseTransformation(str)))

Good catch, thanks. I'll happily accept a fix, otherwise will try to get it in this weekend.

Please review my first PR #860 It should solve this issue.

Resolved by #860.


Onboard B2B feature related integration tests

$subject
Related Issues

https://github.com/wso2/product-is/issues/16359

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

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


Inconsistent results between online and offline LORA model inference

Description:
There is an issue where the inference results obtained using the base model (unsloth/gemma-7b-it-bnb-4bit) differ depending on whether the model is loaded directly from the Hugbing Face Hub or from a local copy in an offline environment.
Steps to Reproduce - Offline (Local) Loading:

Download the base model (unsloth/gemma-7b-it-bnb-4bit) from the Hugging Face Hub and save it in a folder named "unsloth/gemma-7b-it-bnb-4bit".
Disconnect the network to ensure the model is loaded from the locally saved copy (optional)
Load the trained PEFT model from the local folder.
Generate outputs using the locally loaded model.

Steps to Reproduce - Online (Hugging Face Hub) Loading:

Reconnect the network and remove the local copy of the base model (to ensure base model is loaded directly from the Hugging Face Hub)
Compare the generated outputs from the locally loaded model with those obtained when loading the base model directly from the Hugging Face Hub.

Code Used:
In both cases model was loaded like this:
model, tokenizer = FastLanguageModel.from_pretrained(
    model_name = "lora_model2",
    max_seq_length = max_seq_length,
    dtype = dtype,
    load_in_4bit = load_in_4bit,
)

Expected Behavior:
The results generated by the locally loaded model should closely match those obtained when loading the base model directly from the Hugging Face Hub, regardless of the environment (offline or online).
Actual Behavior:
The results obtained when loading the base model directly from the Hugging Face Hub while being online and loading the base model offline from a local folder (with the network disconnected) are significantly different. The LoRA model is local in both cases, with the only difference being where the base model is loaded from (hub vs local copy).
Additional Information:

It is suspected that there might be an issue with offline/local loading of the base model and applying the LoRA, causing the discrepancy in results.
Changing the paths of both the local model and the PEFT config adapter did not resolve the issue.

@terraformmachine That is extremely weird! Essentially your saying a downloaded online model has different generation results than an offline model?
I'll see what I can do! I'll check if there are model weight differences - sorry on the issue!


K8S Scanner populates name instead of resource_id in violations table

[x] Which version of Forseti Security you're using (look in google/cloud/security/__init__.py in your installation).
2.0-RC3
[x] Which module(s) (inventory, scanner, enforcer, explain) you're having trouble with.
Scanner -> kubernetes_cluster
[ ] Include errors, log output, and host operating system, including installed packages.
[x] Note how you deployed (via Deployment Manager, local system, etc.).
Deployment Manager

K8S scanner populates the kubernetes_cluster "name" in the violations table resource_id field. The name is not unique and to be consistent with other scanners,  the ke_cluster.resource_id should be used.

KE is an exception when it comes to resource_ids. Since there is no unique KE cluster ID in GCP, for GCP inventory, Forseti creates a unique id by taking a hash of the KE cluster selflink (services/inventory/base/resource.py:839-840). As such, this ID is not something that can be used outside Forseti to map to a KE cluster in GCP. The current behavior of putting the name - which is not unique - is probably better than the alternatives, although this is a departure from other scanners.


[#154459087] passing through mentions string received from ChatInputToolbar

Approved in previous PR. Incorrect base for the last merge.


[Backport release-22.11] kernel updates

Description of changes
Picked https://github.com/NixOS/nixpkgs/pull/209050 except for linux_testing, as our config can't handle 6.2 even on nixpkgs master yet.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Built on x86_64-linux (will edit): linux_6_0

Hehe
nix-repl> linux_testing_hardened
error: linux_testing_hardened has been removed, please use linux_latest_hardened

nix-repl> linux_latest_hardened
error: The attribute `linuxPackages_hardened_latest' was dropped because [...]

Well, if hardened 6.0 exists there, perhaps it's not so important to test it?

Thx for taking care of that! Skiing this week, so I haven't found the time, next week I'm happy to do kernel updates again :)

I am not fully sure how https://utcc.utoronto.ca/~cks/space/blog/linux/KernelBindBugIn6016 relates into this update for us

The article writes it's present in 6.0.16 and 6.0.18.  From that I expect that this 6.0 bump won't change it.

I suspect it's bad for 6.0, as 6.0.19 was already released as EOL, and based on recency of some posts (and a glance into changelog) I don't expect it's fixed in there.
I think I recently saw some test regression on Hydra that mentioned expecting and not getting EADDRINUSE.

The 6.0 bug was introduced in 6.0.16 and has no fix so far because it is EOL.

Yeah we shod drop 6.0 anyways.

That makes me sad, as connected ethernet immediately wakes up suspend for me on other kernels that I've tried so far.


Do Not Review, Do Not Merge: Prototype for single file PSA/Legacy Mbed TLS configs

@frkv Are you going to pursue the solution here?

@frkv Are you going to pursue the solution here?

Will be discussing this with you offline


Added test for CMUArctic Dataset

PR linked to #821
I have used the same dummy utterance for all the emulated samples.
Requested review: @mthrok

Could you rebase onto the latest master?