btcd mishandles witness size checking #1

https://github.com/DefiantLabs/cosmos-tax-cli/security/dependabot/1

@esaari1 or @KyleMoser can we update https://github.com/DefiantLabs/cosmos-tax-cli/blob/main/go.mod#L43 to this new version?

@KyleMoser can you try this? I updated version to 0.23.2. Get an error cause the subpackage btcec is not being downloaded. Get the same problem when I tried 0.23.4, which is the latest version.
Looking on github.com, I do not even see anything newer that v0.22.1 for btcd. But there is a 0.23.4 if you look at
https://pkg.go.dev/github.com/btcsuite/btcd
and there you can see btcec. So I am not sure what is going on here with go mod.

yeah, I would say try to get it to build with the patched version of btcd.

So we cannot do this just yet. cosmos-sdk depends on btcd and does not have an update yet that is compatible with btcd version 2.32.x.

@esaari1 thanks for looking into this.  We can move this back to todo, and try again later once the update is available.


Add CHANGELOG.md

Add a CHANGELOG.md file, following rules of https://keepachangelog.com/en/1.0.0/
OFC we will need to adjust contribution rules, to force everyone to add changes description to "Unreleased" group on every PR.

This is good idea! We can start by combining the Release notes with Upgrade notes

Anybody willing to start? :)

@pkarw I would like to start this!

Thats great! Please go on and propose the initial version as a Pull Request!


Fix deprecation warning message in Mongoose and connect-mongo.

This PR will resolve #739

This does fix the issue. Thumbs up.

Thank you again for the quick response and the PR submission.  It looks like the problem was because of an issue with connect-mongo which got fixed in its 2.0 release.  I put additional details in the related github issue.
Thanks again.


Deleting an app at the project level after deploying via multicluster app leaves a link to the target in the multicluster app

rancher/rancher:master 1/25
Deleting an app at the project level after deploying via multicluster app leaves a link to the target in the multicluster app.
To reproduce:

Deploy a multicluster app to multiple targets
Delete the corresponding app at project level
View the multicluster app at Global level

Expected:

The multicluster app only has a link to the remaining targets

Actual:

The multicluster app has links to all original targets, and clicking the removed target link shows an error

rancher/rancher:master 2/1
Users are no longer able to delete multicluster apps at the project level. Checked via UI and API.


Would the GPU execution time will restart? ?

I have been also facing the problem I trained my model halfway yesterday and thought the remaining GPU time would restart today as well. After yesterday's training, I had only 45 mins left for GPU usage. Will it get renewed today?
As I need more time to train the model efficiently and then validate and predict using my model?

as i know is 24 hours after u started it, just write what time u started it

Thank you @kadalio3 , the GPU time restarts after 24 hours. If you cannot get next 4 hours despte 24 hours, please let us know.

Thanks it worked . Though instead of 24 hours I had to wait 26 hours.


Create and teardown receivers on the fly, per smoke test, so we can smoke in parallel

Our smoke tests take forever (18 minutes, in Staging, less local but more painful as you wait and wait.
There are a number of things we can do to fix this.  This is one of them.
The big reason why you can't just launch each test in a separate thread, and run in parallel, is that the batching step in ReportStream will grab whatever reports have been created in the last minute, and batch them.  So if you are running tests in parallel, it'll grab data from all kinda different tests, and all-too-cleverly munge them together into a batch, making it look like your tests failed.
So, this story is to work around that:

For each test that submits data to the api/reports or api/waters endpoint, create a setup and teardown step that creates a uniquely named receiver setting for just that test only, then removes that receiver when the test is completed (regardless of whether the test passed or failed)
There is an example of how to do this in WatersAuthTests.kt -- see methods setup() and teardown() there.  However, that sets up a sender, and you are supposed to set up receivers, and a bunch of them (details depend on each test)
Make sure the receiver names are useful and human readable, but won't clash with each other.  eg, "END2END_HL7_BATCH".
Note the kluge that the county names in the test MUST match the receiver names.
I think its ok to assume only one suite of smoke tests is running at a time.  (That is, you don't need to attach a UUID or a timestamp to the receiver name)
In Staging, you'll need to be authenticated to access.

OBE


TLCC Code base: Duplicated in C/C++

This is the current pragmatic state. Obviously has as challenge to keep TLCC code consistent with fabric code. #276 would be more long-term approach

Currently done, so closing this

Re-open to make MVP decision explicit (even though this is status-quo)


TBD: clean out tapi from repo

Now that tbd writer/reader is in llvm, theres no reason to copy/paste the code in swift.

Removed code and dependecies that swift has to generate tbd files, to instead directly call from LLVM.
Note this PR is reliant on commits that are in apple/swift-llvm in the upstream-with-swift branch. Specifically commits

95e7e916d4ac7faac060236e297d2c7bdad32dc3
da9c5afc197c0f6a14dabc5f6170fe4a215ff426

To be able to build and test these changes I cherry-picked those commits to the latest apple/stable branch within apple/swift-llvm and master-rebranch within apple/swift, which is why my PR is against said branch.

fix radar 55129229

Yessssssss, thank you @cyndyishida!!!

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 0cafc21b4b17044351b889f23fc51b8d0d6d4cba

Build failed
Swift Test Linux Platform
Git Sha - 0cafc21b4b17044351b889f23fc51b8d0d6d4cba

PR Testing is disabled for master-rebranch, so I'm going to merge this in, but continue to watch
https://ci.swift.org/view/swift-master-rebranch-stable-20190619/job/oss-swift-master-rebranch-incremental-RA-osx/ for any regressions this commit could have introduced.

We are seeing following test failures:
06:02:10 ********************
06:02:10 Failing Tests (14):
06:02:10     Swift(linux-x86_64) :: reproducible-builds/swiftc-emit-tbd.swift
06:02:10     Swift(linux-x86_64) :: TBD/enum.swift
06:02:10     Swift(linux-x86_64) :: TBD/struct.swift
06:02:10     Swift(linux-x86_64) :: TBD/main.swift
06:02:10     Swift(linux-x86_64) :: TBD/dylib-version.swift
06:02:10     Swift(linux-x86_64) :: TBD/protocol.swift
06:02:10     Swift(linux-x86_64) :: TBD/app-extension.swift
06:02:10     Swift(linux-x86_64) :: TBD/global.swift
06:02:10     Swift(linux-x86_64) :: TBD/function.swift
06:02:10     Swift(linux-x86_64) :: TBD/output-path-deduction.swift
06:02:10     Swift(linux-x86_64) :: TBD/class.swift
06:02:10     Swift(linux-x86_64) :: TBD/specialization.swift
06:02:10     Swift(linux-x86_64) :: TBD/opaque_result_type.swift
06:02:10     Swift(linux-x86_64) :: Runtime/backtrace.swift

on Ubuntu 18.04.
Also, you should be able to request PR testing if you see any issue please let me know.


rvalue-ref qualified accessors for value

as_string et al. should have rvalue-ref qualified overload.

What exactly does this do?

value foo();
void bar(string)

bar(foo().as_string());

This will copy the string. With an rvalue-ref overload it wouldn't.


update notes_tie.lua

Slightly trepidatious, amending plugindef() to fix multi-line error and incorporate GroupName and GroupDescription.
This still doesn't seem to be behaving properly, even though GitHub now tells me my fork is up to date.

Hmmm - I'm wondering if maybe I reverse all this, get back to an online fork that GIthub says is clean, and THEN start a new local fork of the "clean" version?

Maybe that'll help?
Closing again like before.


Cannot create a "custom" OIDC CC application

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Thank you for opening an issue. Please note that we try to keep the Terraform issue tracker reserved for bug reports and feature requests. For general usage questions, please see: https://www.terraform.io/community.html.
PingOne Terraform provider Version

0.5.0
Terraform Version

n/a
Affected Resource(s)

pingone_application

Terraform Configuration Files
# Copy-paste your PingOne related Terraform configurations here - for large Terraform configs,
# please use a service like Dropbox and share a link to the ZIP file. For
# security, you can also encrypt the files using our GPG public key.

# Remember to replace any account/customer sensitive information in the configuration before submitting the issue
resource "pingone_application" "services" {
  environment_id = pingone_environment.my_environment.id
  name           = "Services"
  description = "Client Credentials app used by Services"
  enabled        = true

  oidc_options {
    type                        = "CUSTOM"
    grant_types                 = ["CLIENT_CREDENTIALS"]
    token_endpoint_authn_method = "CLIENT_SECRET_BASIC"
  }
}

Debug Output

pingone_application.services: Creating...
╷
│ Error: Error when calling `CreateApplication`: The request could not be completed. One or more validation errors were in the request.
│
│   with pingone_application.services,
│   on main.tf line 526, in resource "pingone_application" "services":
│  526: resource "pingone_application" "services" {
│
│ PingOne Error Details:
│ ID: ***
│ Code: INVALID_DATA
│ Message: The request could not be completed. One or more validation errors were in the request.
│ Details object: [{"code":"INVALID_VALUE","message":"Cannot deserialize value of type
│ `com.pingidentity.p2.application.model.AppType` from String \"CUSTOM\": not one of the values accepted
│ for Enum class: [PING_ONE_ADMIN_CONSOLE, PING_ONE_PORTAL, CUSTOM_APP, SERVICE, WEB_APP,
│ SINGLE_PAGE_APP, NATIVE_APP, TEMPLATE_APP, WORKER, PORTAL_LINK_APP, PING_ONE_SELF_SERVICE] Must be one
│ of [[WEB_APP, NATIVE_APP, SINGLE_PAGE_APP, SERVICE, CUSTOM_APP, WORKER, PING_ONE_SELF_SERVICE,
│ PING_ONE_ADMIN_CONSOLE, PING_ONE_PORTAL, TEMPLATE_APP, PORTAL_LINK_APP]].","target":"type"}]
╵

Panic Output

N/a
Expected Behavior

Custom application created
Actual Behavior

Error on creation - the CUSTOM application type (as validated by the provider) isn't in the list of available application types on the platform
Steps to Reproduce

terraform apply

Important Factoids

References

Requires update of the schema validation, new acceptance tests and documentation update


GDNative library don't exported with project

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Godot 3.0, master branch
Issue description:
Really I don't know is it bug or there is some undocumented behavior.
I am trying to use GDNative library, so I have built dll using godot_headers and cpp_bindings. It works if I run the project from the editor.
But then I try to export this project. I tried windows and android exports. And this .dll (.so in case of android) doesn't came with the package. Looks like a bug. Or maybe I should do something more to get this library packed with the game?
Also there one more thing, It is more likely question for the developers (maybe @reduz). If I will build multiple dlls for different architectures (now GDNativeLibrary shows me 9 variants) how it will be (or should be) packed with the game? More on this, GDNativeLibrary has only one selection for android, but android can use at least 4 different architectures.

I haven't looked into exporting much yet, so it's not implemented yet. It's not really a bug, more a missing feature.
I have no idea how Android stuff works, @endragor did some things with GDNative and Android, maybe he can help?

@karroffel Since godot is closer to release, can you revisit this bug?

This should work ™  but needs an actual exporter plugin for all the supported desktop platforms. Android and iOS should already work

Ah I knew I had to do this, forgot.
On Dec 31, 2017 1:46 PM, "Thomas Herzog"<EMAIL_ADDRESS>wrote:

This should work ™ but needs an actual exporter plugin for all the
supported desktop platforms. Android and iOS should already work
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/11441#issuecomment-354613742,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2_-5Z6pVDLFlO9lzEzuevQFQyrwNks5tF7ppgaJpZM4Pd6de
.

I've tried with iOS, GDNative library didn't export along with the project. I did manually link .dylib into Xcode project following along the similar approach as of this issue but at the end face the same error as shown in the issue linked.
Faced the same issue with exporting on macOS, I have to manually did this as suggested by community.
If engine can take care preparing all those GDNatives and making them ready to be build for supported platforms, this would really cut so much time as users like myself will spend time invest on building GDNative libraries for each platform prior to that.


unread: Fix messages that cannot be marked as read in narrows.

If you visit a narrow that has unread messages on it that aren't part
of the home view (e.g. in a muted stream), then we were never calling
message_util.do_unread_count_updates, and more importantly,
unread.process_loaded_messages on those messages.  As a result, they
would be unread, and moving the cursor over them would never mark
those messages as read (which was visible through the little green
marker never disappearing).
I can't tell whether this fixes #8042 and/or #8236; neither of them
exactly fits the description of this issue unless the PM threads in
question were muted or something, but this does feel related.

Merged, after doing some verification in production with affected users.


Access control

What is this PR for?
Add access-control function
This PR includes

Backend

Add users and application_user_roles tables

view users can view only
edit users can view and edit application
admin users can add/edit/remove other users

Add admin APIs which can control authorities

GET/PATCH/POST roles for each applications

Frontend

Add Admin page to each applications
Add add/edit/remove forms for editing users for application

What type of PR is it?
Feature
What is the issue?
N/A
How should this be tested?
...

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

@sugyan Thank you for your pull request!
Could you add the test for this?

@sugyan The design of testing for authentication is good.
Could you add tests for every endpoints which use an authentication?

@sugyan Hi. Could you modify 1 change and 1 bugfix?

Add notification when "change role" finished.
Enable "deletion checkbox" when auth disabled.

Fixed bugs.

@sugyan I have created two PR for this PR. Please check it.

https://github.com/sugyan/drucker-dashboard/pull/2
https://github.com/sugyan/drucker-dashboard/pull/3

I think it is LGTM.
Finally could you solve the conflict please?

Sorry, I have no idea about the good implementation of APIRequestResultsRenderer/index.tsx...


Confirm email for your form at ****

Describe the bug
it's not clear how to use this package there's no sign in to have an accout to be verfied, as well the email sends you to a page to inform you that there's an issue go back to nothing.
To Reproduce
Steps to reproduce the behavior:

update<EMAIL_ADDRESS>to your email
Click on  submit
boom nothing
error

Expected behavior
there should be a much clear examples f how to be used, is it need's to have an account on the platform where to sign in, what are the requirements

Seems to the project is abandoned

Hi @walidbou6. I hope you are well.
You should use an email validated. Could try again and give me more information if you have any problem?


"Invalid Exclude" warnings in XCode 13.1

Hi,
I've just updated XCode to the most recent version, and I now get some warnings when building my project (which includes SwiftAA via the Swift package manager in XCode):
Invalid Exclude '.../SourcePackages/checkouts/SwiftAA/Sources/AA+/Sources/AA+/AA+.htm': File not found.
Invalid Exclude .../SourcePackages/checkouts/SwiftAA/Sources/AA+/Sources/AA+/naughter.css': File not found.
Invalid Exclude '.../SourcePackages/checkouts/SwiftAA/Sources/AA+/Sources/AA+/CMakeLists.txt': File not found.
It looks like the problem is caused by the Package.swift files referencing files that aren't present in the project any more.
Those are only warnings, so it is not really a problem and the project builds correctly, but it would be nice if they could be silenced :)
Thanks

Hi. Thanks for the report. I agree. :-) I won't have time before Christmas though. But I keep an eye on it.

We're experiencing the same warnings with Xcode 13.2.1:
Invalid Exclude '/Users/me/Library/Developer/Xcode/DerivedData/Junipa-axynmiysuypgzucopudqyvqlpcbk/SourcePackages/checkouts/SwiftAA/Sources/AA+/Sources/AA+/naughter.css': File not found.
Invalid Exclude '/Users/me/Library/Developer/Xcode/DerivedData/Junipa-axynmiysuypgzucopudqyvqlpcbk/SourcePackages/checkouts/SwiftAA/Sources/AA+/Sources/AA+/CMakeLists.txt': File not found.
Invalid Exclude '/Users/me/Library/Developer/Xcode/DerivedData/Junipa-axynmiysuypgzucopudqyvqlpcbk/SourcePackages/checkouts/SwiftAA/Sources/AA+/Sources/AA+/AA+.htm': File not found.

As stated by @Saturnyn, the project builds correctly, but it would be preferable if the warnings could be fixed. Swift PM doesn't currently allow for warnings to be disabled for package dependencies, which is a problem for our project as we have a zero warning policy...

These warnings can be fixed by making the exclude paths relative to the value of path property for the AAplus package, i.e.
.target(
    name: "AAplus",
    dependencies: [],
    path: "Sources/AA+",
    exclude: ["Sources/AA+/naughter.css", "Sources/AA+/CMakeLists.txt", "Sources/AA+/AA+.htm"]

becomes:
.target(
    name: "AAplus",
    dependencies: [],
    path: "Sources/AA+",
    exclude: ["naughter.css", "CMakeLists.txt", "AA+.htm"]

Thanks for the tip! If I understand well, I just need to make this change, and a release, and that's it? Will do this evening (I'm at work right now)

A new tag 2.3.2 is now available. Let me know if it is OK for you. I'd love to hear about a project using SwiftAA (the one mentionned in the README hasn't been refreshed since a long time...)

I updated the lib and can confirm the warning disappeared and everything seems to be working fine.
Thank you :)

Thanks for the tip! If I understand well, I just need to make this change, and a release, and that's it? Will do this evening (I'm at work right now)

I implemented this fix in a fork and it built correctly and passed all unit tests, so you should probably be able to integrate and release.

A new tag 2.3.2 is now available. Let me know if it is OK for you. I'd love to hear about a project using SwiftAA (the one mentionned in the README hasn't been refreshed since a long time...)

Thank you for releasing this fix so quickly. Much appreciated!


Incorrect SMBIOS version check

This macro checks the SMBIOS version, but for example I have version 3.3 and it fails with an error. There was no such problem in previous versions of the library.
https://github.com/brunexgeek/smbios-parser/blob/75c24b7c6a1cbf74afac7225d850db6aeed6e938/smbios.c#L21

Thanks for reporting the problem. The way it's implemented now will definitely return an error for versions above  3.0 .

@Immortalety I merged a fix for the SMBIOS validation into master. Could you try it and see if it solves your issue?

Yes, this problem has been solved now. Thank you for your work!


Jellyfin cannot set https - SycPlay doesn't work

It expects me to give the location of the certificate.
Obviously with boringproxy, the server version has it, not the client.
Things like SycPlay doesn't work via boringproxy.

Actually, boringproxy lets you choose whether the client or the server controls the certs. That's what the "TLS Termination" setting is. The default is server, which means the server will contact Let's Encrypt for a cert automatically. If you set it to "Client", then the client will do that instead and the server will simply pass traffic to the client. If you set it to "Passthrough", then both the server and client won't do any TLS and it will be up to the application (or you) to get a certificate.
What you might try in your situation is set it to "Client", start up a tunnel. That will get you a cert on the client. Then delete the tunnel and recreate it as "Passthrough", and use SyncPlay to select the cert previously created by the client (check the certmagic docs for where the certs are stored by default, or you can pass the -cert-dir option to boringproxy client to tell it where to store them.

Actually, boringproxy lets you choose whether the client or the server controls the certs. That's what the "TLS Termination" setting is. The default is server, which means the server will contact Let's Encrypt for a cert automatically. If you set it to "Client", then the client will do that instead and the server will simply pass traffic to the client. If you set it to "Passthrough", then both the server and client won't do any TLS and it will be up to the application (or you) to get a certificate.
What you might try in your situation is set it to "Client", start up a tunnel. That will get you a cert on the client. Then delete the tunnel and recreate it as "Passthrough", and use SyncPlay to select the cert previously created by the client (check the certmagic docs for where the certs are stored by default, or you can pass the -cert-dir option to boringproxy client to tell it where to store them.

Going to close for now. Let me know if you have more details or questions.

Going to close for now. Let me know if you have more details or questions.


Discussion: Microsoft.Data.Sqlite 2.0 to target .NET Standard 2.0

Discussion thread for https://github.com/aspnet/Announcements/issues/247.

I take it you will support linux-arm?

In 2.0.0 we leverage SQLitePCL.raw (also used by Azure Mobile, SQLite-net, and the AWS SDK) which supports a very large matrix of platforms out of the box and  lets you easily use your own or the system-wide version of libsqlite3.so.

Ok. How do you decide when to assume os dist presence  and when to include native dependencies like libuv.so? Also, is there a list of assumed os dependencies available to simplify life for meta dist creators and dist creators?

The Microsoft.Data.Sqlite package uses SQLitePCLRaw.bundle_green which will bring its own copy of the native SQLite library on all platforms except iOS where it will use the system's.
The Microsoft.Data.Sqlite.Core package lets you decide. See SQLitePCL.raw's README for all the gritty details on how to specify what native library to use on which platforms.

Ok.  Great😊

@divega The NetCore documentation says In general, we recommend you to target the lowest version of .NET Standard possible. Why does Microsoft not follow their own recommendation? It is fine to increase the version of NetStandard if you want to use some new APIs. But what new API do you want to use with Microsoft.Data.Sqlite?

Primarily Thread.Sleep(). Without it, we can't yield execution in the "table locked" retry loop leading to unnecessary CPU spikes.

Also, now that Visual Studio 2017 15.3 Preview is out with core tools 2.0.0 preview, which supports netstandard2.0 where System.Transactions is implemented, I think Microsoft.Data.Sqlite should support it.
Considering that System.Data.Sqlite supports System.Transactions but does not support netstandard, currently there is no library that supports both netstandard and ambient transactions.

Issue #294 is tracking support for System.Transactions.

I think this is a bad idea. NET45 is an important minimum target for several reasons.

NET451 Its the newest version that ships with Win7 and later.
NET451 is the minimum version that many other Microsoft nuget packages target - aspnetcore is a big one that I know if.
NET45 the version that the Xamarin.Mac "Full Framework" is pegged at.

We will not ship a Windows product targeting above NET451 and it looks like a lot of other software is in the same position.
I've forked the dev branch and "backported" it to NET45 and netstandard1.3, see https://github.com/PingmanTools/Microsoft.Data.Sqlite/commits/backport
Although it did not take many changes and all unit tests pass.. In my opinion the reasons given for netstandard2.0 minimum don't really hold up.

The "backport" fork is now being built and tested https://ci.appveyor.com/project/zone117x/microsoft-data-sqlite
And is published on nuget as Microsoft.Data.Sqlite.Core.Backport https://www.nuget.org/packages/Microsoft.Data.Sqlite.Core.Backport/
Please consider backing out of this decision.

In general I very much like the idea to build just netstandard2.0 library.
The only problem is that currently, mainly on non-Windows-environment the Mono seems to not be fully compatible with .NET Standard 2.0, so I need to use the Microsoft.Data.Sqlite.Core.Backport. Problem with that solution is that I would like my single source code to work with all the platforms (which is the idea of the .Net Standard) but as the Microsoft.Data.SQLite API evolves and Microsoft.Data.Sqlite.Core.Backport is not keeping up, I have to create different solution for Mono. So the idea of the .Net Standard is not working, at least not yet. So I think the decision is good but too early.


Superagent with Webpack

I recently tried to use the browser version of superagent located in superagent/lib/client, but got this error:
Emitter(Request.prototype);
^
TypeError: Emitter is not a function

Looking in the package.json, the browser field is set to map emitter -> component emitter:
"browser": {
    "emitter": "component-emitter"
}

But for some reason, this wasn't working with my webpack build. Maybe this was an issue specific to my build, but replacing require('emitter') to require('component-emitter') resolved my problem.
I originally posted this issue to superagent, and was directed here.

@hoodsy I didn't need to add any webpack-specifc configuration to make superagent work.
My modules just use
import superagent from 'superagent'

@thuringia Trying that, I get request is not defined

@hoodsy How are you calling superagent? We've decided to import the superagent function as 'superagent' in our project, however most examples in the documentation require the module as
const request = require('superagent');

Are you sure it's not a naming error in your code?

Still relevant? Feel free to re-open if you repro. I tried superagent a while ago myself and it worked without any hiccups.


set Sauce Lab credentials environment variables from npm config params

Hey guys,
Thank you for accepting my pull request earlier !
However, since then, I learned that the Sauce Labs environment variables were automatically picked up by WebDriver. Since my changes removed those variables, users will not be able to execute tests using Sauce Labs.
This pull requests fixes this issue by re-setting those environment variables from the npm config parameters.
Sorry about the bug I introduced, and thank you again for those great samples.

makes sense, forgot WD.js automatically picked up those env vars!


Layout changed

Current Behavior
Current layout is less pretty than it used to be in previous releases

Expected Behavior
Please revert to the old lay out, options to make it configurable (css/own logo) would be awesome but not required.
Steps To Reproduce
No response
Environment

openvpn-auth-oauth2 Version:
OpenVPN Server Version:
Server OS:
OpenVPN Client (flavor, OS):
OIDC Provider:

openvpn-auth-oauth2 logs
No response
openvpn server logs
No response
Anything else?
No response

I guess regression introduced in https://github.com/jkroepke/openvpn-auth-oauth2/pull/157


"The window has crashed" on "Open Folder"

VSCode Version: 1.19.0-insider
OS Version: Windows 10

Steps to Reproduce:

Go to File > Open Folder
Select a folder
Observe that the window crashes (see screenshot below)

Clicking "Reopen" will simply cause the window to crash again. If you press "close" and re-launch VS Code, the project will have been opened.

Reproduces without extensions: Yes

Here is a checklist to find out more about the freeze/slow/crash

[ ] it reproduces with extensions disabled (code --disable-extensions from the command line - NOT the integrated terminal of Code)

if it does not reproduce with extensions disabled, you can stop testing and report this!

[ ] it was not happening in the previous stable release
[ ] it reproduces in insiders builds (get it from here)
[ ] there is no suspicious output when running from the command line (code --verbose)
[ ] there is no suspicious output in the window itself (Help | Toggle Developer Tools | Console)
[ ] it reproduces on Windows, Linux and Mac
[ ] it reproduces after deleting <appdata>/User/keybindings.json and <appdata>/User/settings.json (backup first! see below for paths)
[ ] it reproduces when opening empty (code -n from the command line)
[ ] it reproduces opening any folder
[ ] it reproduces opening any file
[ ] it reproduces when only having a single window open
[ ] it reproduces by just using Code without external tools running on the same folder
[ ] it reproduces from a workspace that is not under Git version control
[ ] it reproduces independently if a file is opened or not after startup
[ ] it reproduces when running with GPU disabled (code --disable-gpu)
[ ] it reproduces when being offline
[ ] it reproduces when disabling the file watching (set files.watcherExclude to "**/**": true)
[ ] it reproduces after uninstalling and reinstalling
[ ] it reproduces after deleting the application data directory (backup first! see below for paths)
[ ] I have not played around with permissions or ran Code as administrator
[ ] I am seeing a single CPU process spin high (please share the full command line arguments)
[ ] I am seeing Code consuming lots of memory in a short time

Application Data Directory for VS Code:

Windows: C:\Users\<user>\AppData\Roaming\Code
Mac: /Users/<user>/Library/Application Support/Code
Linux: /home/<user>/.config/Code

Application Data Directory for VS Code Insiders:

Windows: C:\Users\<user>\AppData\Roaming\Code - Insiders
Mac: /Users/<user>/Library/Application Support/Code - Insiders
Linux: /home/<user>/.config/Code - Insiders


Potential Optmization for GroupBy Sorting Phase

Description

Currently, we're using heap sort at the end of groupBy, whose big O time complexity is n+nlogk. Since it is only necessary to keep the number of records up to TRIM_SIZE (normally 5000), we can use the pivot selection algorithm to select topk elements. When the number of records is relatively low (i.e. smaller than 150k), pivot selection algorithm can boost the performance by around 30-40%, at the expanse of extra memory usage. However, current benchmark results show that this algorithm becomes super inefficient if the memory usage exceeds some limits, mainly because of GC overhead. The detailed results and discussion can be found in this link.
Note that this PR does not change the original API and method, but just brings in a second option for the groupBy sorting phase.
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Is this same as standard order statistics algorithm which gives worst case linear time (compared to heap sort approach) for selecting k largest or smallest elements?

Is this same as standard order statistics algorithm which gives worst case linear time (compared to heap sort approach) for selecting k largest or smallest elements?

The selection algorithm used here is quickselct, a partial quicksort algorithm. The best case and average performance is O(n) but the worst case is O(n^2).

Codecov Report

Merging #6875 (148673a) into master (3e20979) will decrease coverage by 8.53%.
The diff coverage is 34.66%.

@@             Coverage Diff              @@
##             master    #6875      +/-   ##
============================================
- Coverage     73.99%   65.45%   -8.54%
  Complexity       12       12
============================================
  Files          1421     1421
  Lines         69141    70055     +914
  Branches       9986    10130     +144
============================================
- Hits          51159    45857    -5302
- Misses        14633    20913    +6280
+ Partials       3349     3285      -64

Flag
Coverage Δ
Complexity Δ

integration
?
?

unittests
65.45% <34.66%> (-0.52%)
12.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


OpenErp Java API uses defunct XMLRPC which has security vulnerabilities

XMLRPC is an archived project at apache: https://ws.apache.org/
XMLRPC has multiple security vulnerabilities:
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-5002
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-5003
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-5004
Suggest moving to another library.

Hi @mbatchelor ,
Thanks for your tips. Do you suggest to move for another XMLRPC client library or to change the API? If you suggested the second option it's not possible, Odoo SA the editor of Odoo is responsible o the technical architecture choices and all its products have been developped based on XMLRPC, neither SOAP nor REST are available in a native way in Odoo.

@mbatchelor,
Ok I understand what you pointed.
BTW, what are the pentaho library you're recommanding, as we hope to be able to migrate the odoo step into the new api using your native xmlrpc library would be a great idea


Figuring out the best method to track web sign ups

@kueda was looking through google web analytics and couldn't determine the best method to track if sign ups are successful. I'm thinking we might consider giving a different url to the first time a viewer looks at the dashboard. Not sure how feasible that is. It would appear Google Web Analytics Goals is perfect for this scenario. However, there also might be better event tracking solutions.
Bottom line... I don't really care what method we use. I just would like to track the percentage of people that click on "Sign Up" and complete the sign up form. Would also be nice to know what page they clicked "Sign Up" from and what other pages they viewed in that session before clicking sign up. Might also be nice to know what they do the first time they see the dashboard, but that is secondary to the act of signing up.

After more research, I think it may be best to remove any parameters after /home. Currently, after completing the sign up flow the url retains the parameter of the screen you were on before entering /signup. For example, “https://www.inaturalist.org/signup?return_to=https://www.inaturalist.org/observations". I am not 100% sure, but if this is something easy to do, it would rule out any doubts I have about tracking the flow of signup to dashboard.
Also, if we could give the /home a different url like /home_first or something for the first time a user views the dashboard, I would be able to track it more easily in Google Analytics. However, let me know if it's something complicated to do.

Man, analytics is kind of brutal. I made it so new users land on /home?new_user=true and updated the goals in Analytics. Let's see if that has any effect.

Tell me about it! Thanks a bunch!!

Joelle, can we close this or is there more you wanted to do? Tracking specific errors in signups seems like it might be a different issue.


Removed pegtl::two and pegtl::three to add pegtl::any_amount

I ran a couple of benchmarks and hence replaced the implementations of two and three by simply using string< C, C > and string< C, C, C > which are equally efficient.
forty_two is only meant as a "fun/joke" rule, so no need to change anything there.
Leaving the question about the generic rules, potentially:

rep_one< N, Cs... > as a short-cut for rep< N, one< Cs... > >.
rep_string< N, Cs... > as a short-cut for rep< N, string< Cs... > >.

The questions are:

How often will this be used/needed in practice? (I can't think of a real-world use-case)
Is there a significantly more efficient implementation available? If not, there is not reason for a new rule - the short-cut alone is not giving enough benefits.

For that reason I added a quick hack of rep_string to the contrib section. It might be more efficient, but I don't think it is worth the effort of making it an official rule.
Feel free to continue the discussion, but for now I consider the matter resolved.


Open to using const for constant references?

In pieces of code such as https://github.com/bcoe/nyc/blob/master/lib/source-map-cache.js#L105, nyc has sets of references at the top of a function, all declared with the var :key:word. However, not all of the references are intended to be variable. In the case of SourceMapCache.prototype._rewriteFunctions, we have f, fnMap & index, where index is necessarily variable and it's two siblings are essentially constant.
:information_source: I find that is becomes faster for everyone, especially newcomers, to read the code when declarations have as much semantic power :zap:   as possible. Using const adds semantic power :zap:  because tells someone that they can rely on the reference to be stable for the rest of it's lifetime :+1:. const also has a low barrier to entry because it's been supported in Node.js for ages (certainly for each of the version in the Travis YAML file) & it's common in many other programming languages. I've provided some additional thoughts & research on const here.
So, are you, the core contributors to NYC, open to using const in modules in the lib folder? I can do the initial legwork including providing some contribution notes regarding use of const. There seems to be sufficient tests to ensure that the behavior of nyc remains constant :musical_score: :clap:.
**It gets reassigned for each key in fileReport.fnMap

@jameswomack I'm happy to entertain this change, and would love the help :+1:
I don't think we're concerned about supporting all the way back to 0.8, @jamestalmage and @isaacs might want to way in on this? I don't thin tap is making an attempt to support 0.8 any more?

I do still support 0.8, since the npm cli has to work there.  But const was supported in 0.8, wasn't it?
$ nave use 0.8 node -e 'const foo = "bar";console.log(foo);foo=1;console.log(foo)'
Already installed: 0.8.28
using 0.8.28
bar
bar

@isaacs gotcha, it existed prior to 0.10 but wasn't announced as a feature?

No idea!  I never bother to read feature announcements, because I always forget and have to check anyway.

@bcoe I've used const in 0.8 w/o issues as well. Maybe I can open a PR w/ proposed changes on a subset of the project?

@jameswomack happy to accept that pull.

I've started moving files over to const 👍

@bcoe awesome


Load multiple NFTMarker

hi. how can i load a multiple nftmarker in nft_threejs.html and nft_worker_threejs.html
I encountered errors when i tried to add multiple nftmarker, encountered below error:
artoolkit.js:2500 Uncaught 20966600 - Exception catching is disabled, this exception cannot be caught. Compile with -s DISABLE_EXCEPTION_CATCHING=0 or DISABLE_EXCEPTION_CATCHING=2 to catch.

sorry nft marker is not supported in ar.js. what you are running is experimental code.


Fix record type generation for simple type references

Purpose

Fixing the invalid record type generation when simple types are referred.
Fixes https://github.com/ballerina-platform/ballerina-openapi/issues/594

Goals
Improving the OpenAPI Tool
Release note
Sample OpenAPI definition
components:
  schemas:
    PetName:
      type: string
    DogName:
      $ref: "#/components/schemas/PetName"
    MyPetName:
      $ref: "#/components/schemas/DogName"

Generated Ballerina types after fixing
public type DogName PetName;

public type PetName string;

public type MyPetName DogName;

Automation tests

Unit tests

Done

Integration tests

Done

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment

JDK 11, Ballerina Swan Lake beta3

Shall we add the test cases for this fix?

Seems this approach does not support following scenario
Pet:
      required:
        - id
        - name
      properties:
        id:
          type: integer
          format: int64
          readOnly: true
        name:
          type: string
        tag:
          type: string
        type:
          type: string
    Dog:
      $ref: "#/components/schemas/Pet"

Generated Ballerina type
public type Pet record {
    int id;
    string name;
    string tag?;
    string 'type?;
};

public type Dog record {
   *Pet;
};

Expected Ballerina type
public type Pet record {
    int id;
    string name;
    string tag?;
    string 'type?;
};

public type Dog Pet;

Since Dog schema is directly referring to the Pet schema we can define Dog type as a type with simple name reference.
@lnash94 Please confirm.

+1 Yes if some schema references to simple record without adding allOf any edge scenarios, IMO it also needs to have  type without type inclusion.

@lnash94 Added a new test case


dump works, but restore not

Output of restic version
restic 0.9.6 compiled with go1.13.6 on linux/amd64
How did you run restic exactly?
I tried to restore the following file out of my snapshots: /etc/caddy/Caddyfile. The snapshots
are taken via restic backup --one-file-system /. I tried restoring the file on two ways:

restic restore latest --target / --path "/etc/caddy/Caddyfile"
restic dump latest etc/caddy/Caddyfile > /etc/caddy/Caddyfile

The latter worked, the first not. Can you explain why? Is this a bug? I've also tried different variations on restic restore. For example: restic restore latest --target /tmp --path "etc/caddy/Caddyfile. I always get the same error:
repository 8b027187 opened successfully, password is correct
latest snapshot for criteria not found: no snapshot found Paths:[etc/caddy/Caddyfile] Host:

What backend/server/service did you use to store the repository?
S3 Backend stored on wasabi.com
Expected behavior

I expect that restic restore works, when restic dump works.
Actual behavior

restic restore fails while restic dump works.
Steps to reproduce the behavior

Do you have any idea what may have caused this?
Is this a path issue? Does dump and restore handle paths differently?
Do you have an idea how to solve the issue?
No.
Did restic help you today? Did it make you happy in any way?

Well, I really like restic. I had to access some backups today, because one of my servers crashed. I just downloaded restic + config and everything worked out of the box. This is cool. The bad part is: I couldn't get a full system restore working, so I just did a rebuild of my server and now I try to get the server up again via dumping single files out of my backup and moving them at the right place.
It would be cool if you could add some documentation about full system backups + restore(!) of linux systems.

You've got the wrong flags for restore. --path is used to select snapshots based on their backup paths. In order to select which files should be restore, you need the --include option.

Closing this issue as this seems to a usage problem, but we got not feedback on the proposed solution.
@shibumi Feel free to comment on this issue if your problem persists.


[Priority: Medium] Change ports in a conscious way

Replaces: https://github.com/stratisproject/Breeze/issues/79
I propose Breeze's ports to be chosen within a long range of unassigned IANA ports, based on this statistics, but also checked against Service Name and Transport Protocol Port Number Registry to avoid port collision.
A reference of common local ports currently used by Breeze and related softwares:

Port
Application

37120
HiddenWallet API

37121
Tor socks port used by HiddenWallet

37122
Tor control port used by HiddenWallet

37123
NTumbleBit server

37124
Tor socks port used by NTumbleBit

37125
Tor control port used by NTumbleBit

9050
Default Tor socks port

9051
Default Tor control port

9150
Tor socks port used by Tor Browser

9151
Tor control port used by Tor Browser

8333
Bitcoin Core Mainnet RPC

18333
Bitcoin Core Testnet RPC

18444
Bitcoin Core Regtest RPC

5000
Stratis: Bitcoin node and Breeze Wallet API

5105
Stratis: Stratis node and Stratis Wallet API

Yeah every ASP.NEt Core app is using port 5000 so having the Bitcoin node on this port is prob not the best idea.
I don't think it's a high priority at the moment though.
I guess we'll can at some point just replace all instances of 5000 and 5105 in our code by new values.


FileNotFoundError: File b'training_samples_X4_1Y.csv' does not exist

First ,it is a great job!
When I train the model, "python3 train.py -arch scd -b 512 -e 300 -op Adam" . The result is "FileNotFoundError: File b'training_samples_X4_1Y.csv' does not exist". Where can I download the file "samples_X4_1Y.csv"?  Where can I download the dataset? Thanks!

Hey, thank you! We're sorry for the lacking file, I added it and made sure the training goes as expected.
Let us know how it goes!
PS : The default batch size gives good results but you could reduce it if you don't have enough memory.


Optimized start up to read directroy and load features async

This won't solve all our startup woes, but this combined with some changes I made in atom-yeoman reduced my startup time from 8 seconds to between 4 and 5 seconds.  So it's nothing to sneeze at.

Looks good to me


NIFI-6692: Fixing NullPointerException when not using Kerberos

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
A KerberosCredentialService is not required by the AbstractKuduProcessor, but if it is not set, a null pointer exception is thrown. The change mirrors the KuduLookupService.
https://issues.apache.org/jira/browse/NIFI-6692
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

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

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

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

[ X ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ X ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ X ] Have you verified that the full build is successful on both JDK 8 and JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

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

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

👍 LGTM. Ideally there would be a test validating this, but I think it's okay to add it later when unifying the Kudu client building logic.

Build failed due to the test: "org.apache.nifi.processors.standard.util.TestSFTPTransferWithSSHTestServer" failing.

Merged to master, thanks @SamHjelmfelt


Consider matching Azure DevOps icons for some subcommands

The Azure DevOps plugin covers 3 different product areas but all as one single plugin. As such, it would be more intuitive to learn and use the Azure DevOps plugin if the auto-complete icons for Pipelines, Repos and Boards matched the in-product icons.
Replace the icon for the 3 subcommands below with the icons used in the product.

I think it is possible to have unique icons defined. As per dev, the data structure does support this.

@DHaussermann As we understand it we think you are asking to change the icon for subcommands board, repos, and pipelines with their respective product icons in the below slash command list

But we could not make this change, as for now there isn't support for that to do through the plugin, I tried reaching out to devs on the community also you can refer to this conversation for more details https://community.mattermost.com/core/pl/467mf9e1wfruxdqwo47frisbuc
Please let us know if we can close this issue or if there is anything else to do here.

Ah I see @avas27JTG. Yes this is what I was wondering about. I was under the impression it may be possible. Thanks for checking.

This cannot be implemented. The plugin framework does not allow for customizing the icons of individual subcommands.
Closing.


fix(mypy): no implicit optional

Issue #, if available:
Description of changes:
Resolve errors related to the "no_implicit_optional" flag in mypy.ini
Checklist

[X] Meet tenets criteria
[X] Update tests
[X] Update docs
[X] PR title follows conventional commit semantics

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

@heitorlessa  with no_implicit_optional=True we have alot more errors coming through in PyCharm.

Codecov Report

Merging #521 (0027d1b) into develop (27c82d8) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #521   +/-   ##
========================================
  Coverage    99.90%   99.90%
========================================
  Files          107      107
  Lines         4280     4280
  Branches       212      212
========================================
  Hits          4276     4276
  Misses           1        1
  Partials         3        3

Impacted Files
Coverage Δ

aws_lambda_powertools/logging/formatter.py
100.00% <ø> (ø)

aws_lambda_powertools/logging/logger.py
100.00% <ø> (ø)

...da_powertools/utilities/idempotency/idempotency.py
98.57% <ø> (ø)

...wertools/utilities/idempotency/persistence/base.py
99.36% <ø> (ø)

...ambda_powertools/utilities/validation/validator.py
100.00% <ø> (ø)

aws_lambda_powertools/event_handler/api_gateway.py
100.00% <100.00%> (ø)

aws_lambda_powertools/event_handler/appsync.py
100.00% <100.00%> (ø)

aws_lambda_powertools/metrics/base.py
100.00% <100.00%> (ø)

aws_lambda_powertools/metrics/metric.py
100.00% <100.00%> (ø)

aws_lambda_powertools/metrics/metrics.py
100.00% <100.00%> (ø)

... and 5 more

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 27c82d8...0027d1b. Read the comment docs.

I've just tried fixing the Union[Tuple, List] with a Sequence and it worked w/ MyPy -- perhaps I have MyPy misconfigured in PyCharm?
I'll look into other bug now then merge as-is if you're still seeing the issue.
from typing import Any, Callable, Dict, List, Optional, Tuple, Union, Sequence

    def __init__(
        self,
        service: Optional[str] = None,
        disabled: Optional[bool] = None,
        auto_patch: Optional[bool] = None,
        patch_modules: Optional[Sequence[str]] = None,
        provider: Optional[BaseProvider] = None,
    ):

    def patch(self, modules: Optional[Sequence[str]] = None):

    def __build_config(
        self,
        service: Optional[str] = None,
        disabled: Optional[bool] = None,
        auto_patch: Optional[bool] = None,
        patch_modules: Optional[Sequence[str]] = None,
        provider: Optional[BaseProvider] = None,
    ):

I've also added the following modules to ignore their lack of typehints, which makes me wonder how reliable this is: https://pypi.org/project/mypy-boto3/
[mypy-boto3]
ignore_missing_imports = True

[mypy-boto3.dynamodb.conditions]
ignore_missing_imports = True

[mypy-botocore.config]
ignore_missing_imports = True

[mypy-botocore.exceptions]
ignore_missing_imports = True

[mypy-aws_xray_sdk.ext.aiohttp.client]
ignore_missing_imports = True

@heitorlessa ok your suggestions work prefectly, i have pushed the latest


how to make serialization work with slightest page changes?

https://dl.dropboxusercontent.com/u/1410209/work/jaagaa/software/rangy-master/demos/serializer.html
https://dl.dropboxusercontent.com/u/1410209/work/jaagaa/software/rangy-master/demos/serializer-with-small-change.html
generate code from first url and paste it in second url which has really small change in title...it doesn't work...
web pages change every time, say some structure is changed still preserving the same text, such as new AD script added, or AD's blocked or some new sidebar component is added, to solve this problem, does Rangy support such cases? is there any workaround such problems
Appreciate a solution for this problem

The highlighter module has an option to create a highlight scoped to a particular container element, so long as that element has an ID (which is essential for serialization). Otherwise, I can't see an obvious way of dealing with document changes.


xDNSServer: OptIn to Dsc Resource Meta Tests

Pull Request (PR) description
This PR opts in to the following DSC Resource meta tests:

Common Tests - Relative Path Length
Common Tests - Validate Markdown Links
Common Tests - Custom Script Analyzer Rules
Common Tests - Required Script Analyzer Rules
Common Tests - Flagged Script Analyzer Rules

To pass the Common Tests - Custom Script Analyzer Rules test, the following files had formatting fixed:

DSCResources/MSFT_xDnsServerSecondaryZone/MSFT_xDnsServerSecondaryZone.psm1
DSCResources/MSFT_xDnsServerSetting/MSFT_xDnsServerSetting.psm1
DSCResources/MSFT_xDnsServerZoneTransfer/MSFT_xDnsServerZoneTransfer.psm1

To pass the Common Tests - Flagged Script Analyzer Rules test, the following file had verbose messages added:

DSCResources/MSFT_xDnsServerDiagnostics/MSFT_xDnsServerDiagnostics.psm1

This Pull Request (PR) fixes the following issues
None
Task list

[ ] Added an entry under the Unreleased section of the change log in the README.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #106 into dev will decrease coverage by 0.14%.
The diff coverage is 38.46%.

@@            Coverage Diff             @@
##              dev     #106      +/-   ##
==========================================
- Coverage   81.37%   81.22%   -0.15%
==========================================
  Files          15       15
  Lines         875      879       +4
==========================================
+ Hits          712      714       +2
- Misses        163      165       +2

Impacted Files
Coverage Δ

..._xDnsServerZoneScope/MSFT_xDnsServerZoneScope.psm1
100% <ø> (ø)
:arrow_up:

...verSecondaryZone/MSFT_xDnsServerSecondaryZone.psm1
5.12% <0%> (-0.14%)
:arrow_down:

...sServerDiagnostics/MSFT_xDnsServerDiagnostics.psm1
89.13% <100%> (+0.49%)
:arrow_up:

...erverZoneTransfer/MSFT_xDnsServerZoneTransfer.psm1
93.33% <100%> (ø)
:arrow_up:

...MSFT_xDnsServerSetting/MSFT_xDnsServerSetting.psm1
76.47% <100%> (ø)
: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 7266502...75e6f2d. Read the comment docs.

LGTM


Secondary Authentication fails in Production style URL

Which service(blob, file, queue, table) does this issue concern?
Blob, queue, table, all of them fail.
Which version of the Azurite was used?
V3.26.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
NuGet - custom built
What's the Node.js version?
V18 LTS
What problem was encountered?
In Authentication string to sign, Authentication Path is not formed correctly when using Production style URL for secondary requests.
Steps to reproduce the issue?
Take any storage client, use production style URL with -secondary suffix for secondary read operation on Azurite.
You can observe the string to sign is malformed for these requests.
Have you found a mitigation/solution?
Yes, I have fixed the error. Until it gets merged use non-production style URL for secondary requests.

Snippet which causes the regression:
if (isSecondary) {
const pos = blobContext.authenticationPath.search(SECONDARY_SUFFIX);
blobContext.authenticationPath =
blobContext.authenticationPath.substr(0, pos) +
blobContext.authenticationPath.substr(pos + SECONDARY_SUFFIX.length);
}
Now, this works fine for non-production URLs like - <IP_ADDRESS>:10000/devaccount-secondary/testcontainer. As there is a secondary suffix in the authentication path - /devaccount-secondary/testcontainer. So this path gets transformed to /devaccount/testcontainer
Now, let's take a look at production URL trying to access a secondary storage, devaccount-secondary.sample.com/testcontainer.
The authentication path looks like this - /testcontainer.
now, when we search for secondary_suffix, the code would return -1 (non-existence). In the next line, we are doing a substring of (0,-1) and (-1+10) - (9). So, the resulting string would be "ainer". This causes the authentication string to be malformed and results in authentication errors for production style secondary URLs.

@vigneashs
Thanks for finding and investigation of this issue!
Azurite welcome contribution!
Would you like to raise a PR to fix this issue?

Thanks, I have raised the PR - Fixes issue #2208 by vigneashs · Pull Request #2215 · Azure/Azurite (github.com)https://github.com/Azure/Azurite/pull/2215
[https://opengraph.githubassets.com/060d4701f271684bea480c340e5c422ed0e3809ce5c18bd6c3e5ad0396ebf42b/Azure/Azurite/pull/2215]https://github.com/Azure/Azurite/pull/2215
Fixes issue #2208 by vigneashs · Pull Request #2215 · Azure/Azuritehttps://github.com/Azure/Azurite/pull/2215
This PR adds additional validation to authentication path computation for secondary requests.
github.com
Regards,
Vigneash

From: Wei Wei @.>
Sent: Monday, October 9, 2023 7:48 PM
To: Azure/Azurite @.>
Cc: Mention @.>; Author @.>; Comment @.***>
Subject: Re: [Azure/Azurite] Secondary Authentication fails in Production style URL (Issue #2208)
@vigneashshttps://github.com/vigneashs
Thanks for finding and investigation of this issue!
Azurite welcome contribution!
Would you like to raise a PR to fix this issue?
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/Azurite/issues/2208#issuecomment-1754251410 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWIT5VXCDWRHNYHZCJQ5XW3X6SZQHBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVEYTEMRSGY4TQOJSQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRRHEZTGNJRGEZTONNHORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@vigneashs
I have reviewed your PR, and raised a comment for change log and test case.
Would you please help to look?


yarn build leads to Failed to compile

After I have this package, I got:
➜  frontend git:(feature/project-report) ✗ yarn build
yarn run v1.22.4
$ GENERATE_SOURCEMAP=false react-scripts build
Creating an optimized production build...
Failed to compile.

Failed to minify the bundle. Error: static/js/main.24b8b55e.js from UglifyJs
Unexpected token: name (e) [static/js/main.24b8b55e.js:84829,4]
    at /Users/user/gitlab/frontend/node_modules/react-scripts/scripts/build.js:128:23
    at /Users/user/gitlab/frontend/node_modules/webpack/lib/Compiler.js:269:13
    at Compiler.emitRecords (/Users/user/gitlab/frontend/node_modules/webpack/lib/Compiler.js:375:38)
    at /Users/user/gitlab/frontend/node_modules/webpack/lib/Compiler.js:262:10
    at /Users/user/gitlab/frontend/node_modules/webpack/lib/Compiler.js:368:12
    at next (/Users/user/gitlab/frontend/node_modules/tapable/lib/Tapable.js:218:11)
    at Compiler.<anonymous> (/Users/user/gitlab/frontend/node_modules/webpack/lib/performance/SizeLimitsPlugin.js:99:4)
    at next (/Users/user/gitlab/frontend/node_modules/tapable/lib/Tapable.js:220:14)
    at /Users/user/gitlab/frontend/node_modules/sw-precache-webpack-plugin/lib/index.js:98:18
Read more here: http://bit.ly/2tRViJ9

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Delete this package, it works smoothly. Can someone shining some light on this issue?

@ww8912188,
This shouldn't happen can you provide minimal sample?, is it CRA, Nextjs repository

Looking at the error logs says that you might be passing some unnecessary prop

this component does not accept prop "name" if added removing might solve this
but it looks like tapable module is causing this error and this component is not dependent upon it

We could not privide the source code. Sorry about that, anyway.


Live Preview breaks when preview frame context changes

Link to reproduction
No response
Describe the Bug
TL;DR
Live Preview stops functioning when the preview frame changes context, either by navigating (away and back to the same doc) or by refreshing. I can't replicate this behaviour locally, it only occurs when deployed (via Payload Cloud in my case).
Any changes after a context change result in a console warning and an empty data object being returned. Warning:
Payload Live Preview: No `fieldSchemaJSON` was received from the parent window. Unable to merge data.

This occurs both for @payloadcms/live-preview and @payloadcms/live-preview-react, both exhibiting the same issue. This appears to be a Payload bug, rather than anything specific to the front-end.
Description
My Next (app router) site uses the useLivePreview hook to watch for when data.updatedAt changes, then calls router.refresh()- using Next's dynamic rendering via Draft Mode to fetch the new version of the document, rather than hot-swapping out the page data onChange. I've gone with this strategy to avoid having page-level client components - this is a pre-existing Next app we're migrating from a different CMS, and I'd like to avoid structural changes to the front-end's architecture.
To achieve this, I have a <DraftIndicator> client component that renders in app/layout.tsx if Next is in Draft Mode. Live Preview hits an api route to enable Draft Mode. This component uses the following code to watch for changes to data.updatedAt, refreshing when it changes.
const lastUpdatedAt = useRef<string>();
const { data, isLoading } = useLivePreview({
  serverURL: process.env.NEXT_PUBLIC_PAYLOAD_URL ?? "",
  initialData: {} as { [key: string]: any },
});

useEffect(() => {
  if (!lastUpdatedAt.current && data.updatedAt) {
    lastUpdatedAt.current = data.updatedAt;
    return;
  }

  if (data.updatedAt && lastUpdatedAt.current !== data.updatedAt) {
    lastUpdatedAt.current = data.updatedAt;
    console.log("Change in `updatedAt` detected, refreshing page.");
    router.refresh();
  }
}, [data, isLoading]);

I recognise that this is a non-standard and likely unexpected use of the useLivePreview hook, but I believe the underlying bug of frame context changes breaking Payload's Live Preview functionality to still be a valid issue worth resolving. End users will not necessarily intuit that navigating away and then back to the same document would stop the Live Preview from updating.
To Reproduce
I so far have only been able to reproduce this issue in a deployed instance via Payload Cloud. I am unable to reproduce this issue locally

Host a Payload instance using Payload Cloud, with a collection that utilises Live Preview.
Host a front-end that utilises the useLivePreview hook.
Within Payload's admin UI preview frame:

Navigate away and back again
Right-click > "Reload Frame" to refresh the preview frame

Check the console for the following warning:Payload Live Preview: No `fieldSchemaJSON` was received from the parent window. Unable to merge data.

Observe that changed made in the Admin UI are no longer reflected in the preview frame

Payload Version
2.8.2
Adapters and Plugins
@payloadcms/plugin-cloud, @payloadcms/live-preview-react

Attached is a video of the issue. The front-end is logging data every time it updates via useEffect
https://github.com/payloadcms/payload/assets/17864782/5277dc2b-dd27-4d6f-924c-8ffc297aa65b

I have observed that the preview's iframe seems to re-render and hard navigate when hosted (Payload Cloud), which does not happen locally - I believe this might be related. This is true even pointing to the same hosted front-end. This issue seems to only occur when hard navigation takes place.
If Payload is hosted locally, all soft navigation stays soft, and this issue does not occur.
But if Payload is hosted, all soft navigation becomes hard navigation, and this issue occurs.

A workaround that worked in my case was to use a nested iframe. Then even with a reload on navigation on that page would only happen in the nested iframe. and the iframe that payload uses wouldn't navigate.


Support for AOT compilation

With Dart 2.3, you can compile your backend ahead of time but it doesn't work with angel ...
Steps to reproduce:
Download Dart 2.3 here : http://gsdview.appspot.com/dart-archive/channels/stable/raw/2.3.0/sdk/
Create a new angel project with angel init test_project
Go to test_project and run dart2aot bin/prod.dart bin/prod.dart.aot
Problem:
You get this error :
 dart2aot bin/prod.dart bin/prod.dart.aot
error: import of dart:mirrors with --enable-mirrors=false

Cause:
Like the case of Flutter, the package 'mirrors' doesn't work with aot compilation ... No workaround, just find a way to replace it (with reflectable for example which is ok with aot)

Actually, Angel 2 has no dependency on dart:mirrors. Reflection is abstracted away via package:angel_container.
Note the call to MirrorsReflector() in prod.dart. If you delete that, along with the import of package:angel_container:mirrors.dart, you will be able to build it.
Removing mirrors-by-default was one of the biggest changes in Angel 2.
Also, there already is support for using package:reflectable. See package:angel_container_generator.

Thank you for your reply. I'm going to try your advices.

Still failing despite your instruction.
prod.dart
import 'package:dart_hello/dart_hello.dart';
import 'package:angel_production/angel_production.dart';

main(List<String> args) =>
    Runner('dart_hello', configureServer)
        .run(args);

Command :
dart2aot bin/prod.dart bin/prod.dart.aot
error: import of dart:mirrors with --enable-mirrors=false

Ah, on further investigation, I figured it out. angel_jael depends on jael, which uses dart:mirrors.
Just remove the angel_jael dependency, along with the usage of MirrorsReflector(), and it'll work.

Indeed, I commented in config.dart the importation of angel_jael and configuration line, now it works !
Instant startup, reduce memory from about 250mb to 30mb, all good :-)
Thank you for your help.

No problem! Excited.
I might have to release a Jael v3 without mirrors, because templating is
convenient.
That being said, setting up reflectable just for the purposes of running
a templating engine might be overkill.
On Tue, May 7, 2019 at 10:53 AM Sl@ny <EMAIL_ADDRESS>wrote:

Indeed, I commented in config.dart the importation of angel_jael and
configuration line, now it works !
Instant startup, reduce memory from about 250mb to 30mb, all good :-)
Thank you for your help.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/angel-dart/angel/issues/103#issuecomment-490114361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACMIUPH3JX6PTUAKGLNLY6TPUGJWFANCNFSM4HLF2JGA
.

--
Tobe Osakwe
Student, B.S. Comp. Sci, Florida State University
https://thosakwe.com


specify Placement Groups

To maximize network performance between my EC2 machines that will be talking to each other, I need to specify the "Placement Group":
http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/placement-groups.html
Once that's done, then my instances stand to possibly benefit from "Enhanced Networking": http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/enhanced-networking.html  Although I'd need to investigate if the AMI & setup Kontena does is enabled for it.
If maintaining this stuff in the Kontena AWS plugin gets unwieldy, it would at least be nice to document a method in which I could create Kontena nodes manually in EC2.

Partially answering my own question, it appears I can use CoreOS on EC2 and then use Kontena's CoreOS config generator: https://www.kontena.io/docs/getting-started/installing/coreos#installing-kontena-nodes  So in this way I can self-serve instance creation, letting me handle that part.  This should give more control over various things (e.g. instance volumes) wherein I'll customize the CoreOS config Kontena generates.  It's not yet evident if there is a point in doing this in ECS (EC2 Container Service).  Better monitoring perhaps?


Editable colors

Nice library. Would it be possible to make the gear colors as well as the background color customizable?
Thanks

Hi @alexbredy
Yes, it's definitely in my todo list, I just need to find some time to devote to this control. I'll make a note and hope to get some time to work on it later this week. I'll keep you posted.

Ciao @andreamazz. Thanks for your swift reply, looking forward.

Hi. Checkout latest commit for gearTintColor.
Cheers


Unable to assign multiple people to list

For Support Requests: Discord

I'm submitting a ...

[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support request here, check the Support Discord for more information.

What is the current behavior?
Multiple people are not allowed to complete a task list

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem (just attach a screenshot or video)

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

is not worked
/assign command not worked
and when someone clicked on assign reaction , his can join to todo but can not checked todo tasks

Hey @ralsonnara thank you for your bug report on github. If you want you can join the support discord to give me a bit more details


Can't install the MBT via NPM due to Header Check Issues

Hi colleagues,
In my project, I'm installing the mbt through npm and even have it in my ci/cd pipeline installed via npm. Since this morning, in the pipeline the installation is failing. I was able to produce this error locally as well:
><EMAIL_ADDRESS>install C:\Users\xxx\dev\tmp\mbt-test\node_modules\mbt
> binwrap-install

ERR Error decompressing https://github.com/SAP/cloud-mta-build-tool/releases/download/v1.0.15/cloud-mta-build-tool_1.0.15_Windows_amd64.tar.gz Error: incorrect header check

A minimal working example for me was just 4 commands:
mkdir mbt-test
cd mbt-test
npm init --yes

npm i mbt
> failure

As you can see, I'm on Windows 10, but the pipeline is on Linux:
ERR Error decompressing https://github.com/SAP/cloud-mta-build-tool/releases/download/v1.0.15/cloud-mta-build-tool_1.0.15_Linux_amd64.tar.gz Error: incorrect header check

The interesting thing is that the issue also occurs in my MWE, when I install an older version like npm i<EMAIL_ADDRESS>Thank you very much for your help in resolving this!
Kind regards
Lukas

It looks like the issue somehow resolved itself.
Do you know what the root-cause of this was? How can we debug issues like this in the future?

Hi @LukasHeimann, it looks like GitHub was down, see here: https://www.githubstatus.com/incidents/j597fw8kv04c?utm_ts=1594629104

It sounds like temporary issue with npm.
Not related to mbt.


Does not contain all guess words

Tested against the original solutions list, this is missing the solutions that were removed by NY Times:
{'agora', 'fibre', 'lynch', 'pupal', 'slave', 'wench'}
All of these are allowed entries in Wordle. I wonder if anything else is missing?

The correct solution and allowed word lists are in the source code here: https://www.nytimes.com/games/wordle/main.9622bc55.js

Yes, I removed the words that the NYT did in https://github.com/tabatkins/wordle-list/commit/12642a315d92f32366b9e44854074be3cccc6753
I haven't updated to the newer changes because this is an extremely low priority project for me. Feel free to PR.


using with chai-exclude

Is there a way to use chai-as-promised with chai-exclude?
I tried the following:
import 'mocha';
import * as chai from 'chai';
import * as chaiAsPromised from 'chai-as-promised';
import chaiExclude = require('chai-exclude');

chai.should();
chai.use(chaiExclude);
chai.use(chaiAsPromised);
const {expect} = chai;

it("chai-exclude-promised-v1", ()=>{
    return expect(Promise.resolve({ a: 'a', b: 'b' })).excluding('a').to.eventually.deep.equal({ b: 'b' });
});

it("chai-exclude-promised-v2", ()=>{
    return expect(Promise.resolve({ a: 'a', b: 'b' })).to.eventually.excluding('a').deep.equal({ b: 'b' });
});

I'm using:
nodejs: 8.9.0
mocha: 4.0.1
chai: 4.1.2
chai-as-promised: 7.1.1
chai-exclude: 1.0.3

I'm trying to figure out the exact same use-case. Did you manage to find a solution?

In my case the error is pretty self-explanatory:
AssertionError: object tested must be an array, or an object, but promise given
But I haven't got a clue how to fix this myself

The issue is still there. I couldn't find the solution yet. 😕

This work for me:
const {use, expect} = require("chai");
use(require("chai-exclude"));
use(require("chai-as-promised"));

it("testing", function () {
  return expect(Promise.resolve({a: 1, b: 2})).to.eventually.excluding('a').deep.equal({b: 2});
});

Package.json:
  "dependencies": {
    "chai": "^4.1.2",
    "chai-as-promised": "^7.1.1",
    "chai-exclude": "^1.0.7",
    "mocha": "^4.1.0"
  }

@95gabor @meeber order of plugin installation is important ... when I switched order it worked :)
thanks for help!


Support for DisplayLink (USB display adapter or dock)

Before opening the issue, have you...?

[X] Searched for existing issues
[X] I have read through the Monitor-Troubleshooting Wiki
[X] Updated MonitorControl to the latest version (if applicable)

Describe the issue
I have an HDMI monitor (Dell S2315H) connected via a USB dongle (https://www.amazon.com/gp/product/B01CSG7TUC/)
This software works great for my other monitors (USB-C or HDMI direct), but not for this one connected via USB adapter.
In preferences, it shows it is aware of the monitor, but says "No Control Available"
Wondering if it is possible fix?
Expected behavior
Dimming for this monitor.
Anything else?
No response
Environment Information (please complete the following information)
- macOS version: 11.2.3 Bug Sur
- Mac model: Mac Mini M1/2020
- MonitorControl version: 3.0.0 Build 3097
- Monitor(s): Dell S2315H
- Monitor Cable(s)/Connection(s):  https://www.amazon.com/gp/product/B01CSG7TUC and then HDMI
- Apple Silicon/M1 (yes or no): yes

Hi @GeoffreyPlitt
DisplayLink currently does not support DDC/CI on macOS so there is no way to add support for these devices.
You can raise your voice regarding this here: https://support.displaylink.com/forums/287786-displaylink-feature-suggestions/suggestions/16135240-add-brightness-support-to-non-windows-oses
Also, as far as I know DisplayLink drivers are incompatible with gamma table calibration (used for software dimming), but I am not entirely sure about this, more experimentation would be needed with this. MonitorControl currently disables any kind of control for DisplayLink devices.
I convert this issue to a discussion, maybe the situation will change.


Ensure ReferenceCountedOpenSslEngine not swallow the close_notify

Motivation:
We need to ensure we not swallow the close_notify that should be send back to the remote peer. See [#6167]
Modifications:
Only call shutdown() in closeInbound() if there is nothing pending that should be send back to the remote peer.
Result:
close_notify is correctly send back to the remote peer.

Also @markdascher ....

Do we also need to modify status to return SSLEngineResult.HandshakeStatus#WRAP if there is a close_notify pending [1]?
[1] https://github.com/netty/netty/blob/4.1/handler/src/main/java/io/netty/handler/ssl/ReferenceCountedOpenSslEngine.java#L903

@Scottmitch it already returns NEED_WRAP (verified with print statements)

@normanmaurer - is it feasible for to provide a unit test for this case?

@Scottmitch I hope yes... gimme a day and I see if I can make it happen

@Scottmitch the good news is I was able to write a test... that bad is that it showed another "edge-case"... Working on it as we speak ...

@Scottmitch @markdascher PTAL again.. with unit test and everything ;)

Cherry-picked into 4.1 (dd055c0) and 4.0 (bb6b251)


Thermostat tolerance

Just a question : is it possible to set tolerance for thermostats (per device) ?

This depends on the device.  Additional configuration options can be added as number entities if they are available on the device.


add CI for building dockers

What does this PR do?
Fixes at this moment we do not have any test all dockers are buildable and we find it out just at master when the fist build is performed
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Are we going to test this base-cuda build as well?
Right now, it's skipped...
https://github.com/PyTorchLightning/pytorch-lightning/runs/1087774602?check_suite_focus=true

Are we going to test this base-cuda docker build as well?
Right now, it's skipped... https://github.com/PyTorchLightning/pytorch-lightning/runs/1087774602?check_suite_focus=true

good catch, thx :]

@Borda https://github.com/PyTorchLightning/pytorch-lightning/blob/a2d14306f70ec9e2a83351819d9ad4b6de9ddb26/.github/workflows/ci_pt-conda.yml#L66
this needs to change, right?

@tgaddair it seems we have again some problem with installing Horovoed on conda env...

@ydcjeff it seems there is some problem with the Cuda docker (installing Horovod) and as you already have a ready update I ll comment it now and we ll uncomment in your PR


Crash when typing nested compount literals

If I type something like
Rect :: struct {
	topleft: [2]int,
}
t := Rect{{1,}}

ols always crashes after typing ,
The crash is here
https://github.com/DanielGavin/ols/blob/a15c6eaad6d2d6f8af1e6cfb22dc5c59efa5d7ef/src/analysis/analysis.odin#L304
because current_comp_lit is nil

I have made a temporary fix by exiting early if it's nil. It's most likely some issue with either my code or the parser. But I'm stuck without the test system working on Odin. I'm keeping the issue open.

Fixed


when i want to upgrade or migra or backup portus's data,how to do that.

BTW,when the master is available

Upgrade Portus

If you are on openSUSE, then it's a matter of zypper up. Otherwise, you'll have to git pull on the current stable branch (v2.0).

Migrate

It usually involves performing rake db:migrate. If you are using the RPM, then this will be done automatically when restarting apache. If there were any other migration steps, we would document it clearly (on the release notes and on http://port.us.org).

Backup

This is not an issue within Portus itself :) Just use whatever tool you like to perform DB backups.

BTW,when the master is available

What do you mean ?

i mean the master build failing,i can not use it for deploy.

@ghoul008 it is passing.

thanks


fix typo in SOA suite chart

not helm update but helm upgrade
Signed-off : Emmanuel Leroy<EMAIL_ADDRESS>

@Djelibeybi Could you review / merge.
Thank you


channel history is not loaded at startup

Describe your issue

On the latest nightly build, the channel history is not loading at startup even though I have it enabled. I've also tried re-enabling it but that didn't help.
Screenshots

OS and Chatterino Version

Windows 10, latest nightly

I think I should split the problems into two issues. What do you think?

Please keep in mind that loading message history is being pulled from 3rd party service that may not always be available - at the moment of creating the issue (and writing this comment) service experiences degraded perforance and there's nothing we - Chatteirno people can do about this, just wait a bit until the service restarts.

indicating something with the history API is wrong wouldn't be a bad idea, it could possibly prevent issues like this

indicating something with the history API is wrong wouldn't be a bad idea, it could possibly prevent issues like this

Fully agree, I just had a thought about adding at least a system message informing the user about status of the recent-messages request / response. Something similar to responses that you receive upon refreshing third party emotes.


add clear-sky fluxes to leaderboard

Purpose

We don't have rmses from other models for these diagnostics, so only biases plots are added.
To-do

Content

[ ] I have read and checked the items on the review checklist.

looks like this:

Fixed bug in file reading, now looks like this:

AMIP 4 seasons


Implement "yogaya generate" command for Terraform code generation

Develop functionality to generate Terraform configurations from existing cloud resources.
Key Implementation Points:

Integrate Former2 for AWS resource scanning
Implement or integrate solution for GCP resource scanning
Implement or integrate solution for Azure resource scanning
Generate consolidated Terraform code
Handle multiple cloud providers and accounts

Technical Requirements:

Research and integrate Former2 codebase
Identify/develop solutions for GCP/Azure
Implement resource scanning logic
Generate valid Terraform configurations
Handle state management

Acceptance Criteria:

Successfully scans resources from all configured cloud providers
Generates valid Terraform code
Handles errors gracefully
Provides progress feedback during generation
Documentation for generation process

Dependencies: Requires completion of #2

@yogaya-taka let's build yogaya cli as a fork from terraformer project. https://github.com/Yogaya-Inc/yogaya_cli is the fork I recently created.


noter.Editor: Play nice with the system keymap, regardless of what it is.

Use the ebiten.KeyName() and ebiten.AppendInputChars() to properly respect the input keymap.
Tested on Linux with US, Dvorak, and French (Azerty) keymaps.

Works in my tests too!
I didn't know about this feature of Ebiten. Very cool stuff!


Add the thumbnail field to the Princeton config

The thumbnail field is not being harvested from Princeton which causes the transformation task to fail.

This is included in #296


Change golang.org/x/sys version

We as the API consumers are seeing issues related to https://github.com/ory/dockertest/issues/212 when building images for OSX.
This is the temporary fix before we move the API out to a dedicated repo, so that consumer projects don't need to track this dependency.
Command we ran to generate this fix:
go get golang.org/x/sys@f9321e4c35
go mod tidy
go mod vendor

@solcates PTAL
/cc @justinpettit

Now we found another fix without modifying this. Thanks.