Update scan-images-defender.md

Need to update all the links as the original doc from MS Defender for Cloud with Container Restristry vulnerability scan have been deprecated and the current document is not responding to the correct new links. 404 Lost links messages for the original links
New and working links
Learn more about Microsoft Defender for container registries

Learn more about container security in Microsoft Defender for Cloud

Learn Build status updates of commit 6e2d3a9:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/container-registry/scan-images-defender.md
:warning:Warning

Details

articles/container-registry/scan-images-defender.md

Line 14, Column 20: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-va-acr' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 15, Column 20: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-introduction' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 19, Column 179: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-va-acr#view-and-remediate-findings' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 23, Column 18: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-va-acr' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 14, Column 20: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-va-acr' will be broken in isolated environments. Replace with a relative link.
Line 15, Column 20: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-introduction' will be broken in isolated environments. Replace with a relative link.
Line 19, Column 179: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-va-acr#view-and-remediate-findings' will be broken in isolated environments. Replace with a relative link.
Line 23, Column 18: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/azure/defender-for-cloud/defender-for-containers-va-acr' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@tejaswikolli-web
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Possible to disable debug logging?

Recently I'm seeing a ton of extra log output and I don't recall specifying debug logging anywhere. How can I revert this to an info or warn type of level? Sample output:
today at 9:43 PM 2020-10-12 21:43:26,370 - Python Cloudflare API v4 - DEBUG - Response: error 81053 An A, AAAA or CNAME record already exists with that host.
today at 9:43 PM [ERROR] ** php.xxxxxxxx.com - 81053 An A, AAAA or CNAME record already exists with that host.
today at 9:43 PM 2020-10-12 21:43:26,381 - Python Cloudflare API v4 - DEBUG - Call: emulated curl command ...
today at 9:43 PM curl -X GET "https://api.cloudflare.com/client/v4/zones/af7b02f6760430b6eaaf6489002e9251/dns_records?name=redis.xxxxxxxx.com" \
today at 9:43 PM -H "User-Agent: python-cloudflare/2.8.13/python-requests/2.23.0/python/3.8.5" \
today at 9:43 PM -H "Content-Type: application/json" \
today at 9:43 PM -H "X-Auth-Email<EMAIL_ADDRESS>\
today at 9:43 PM -H "X-Auth-Key: REDACTED"

Config:
# Cloudflare-Companion - Automatic container CNAME DNS Creation
  cloudflare-companion:
    image: tiredofit/traefik-cloudflare-companion
    container_name: cloudflare-companion
    restart: unless-stopped
    security_opt:
      - no-new-privileges:true
    networks:
      - $SOCKET_NETWORK
    depends_on:
      - socket-proxy
    environment:
      - TIMEZONE=$TZ
      - TRAEFIK_VERSION=2
      - CF_EMAIL_FILE=/run/secrets/cf_email
      - CF_TOKEN_FILE=/run/secrets/cf_token
      - TARGET_DOMAIN=$DOMAINNAME
      - DOMAIN1=$DOMAINNAME
      - DOMAIN1_ZONE_ID=$CF_API_ZONE_ID
      - DOMAIN1_PROXIED=TRUE
      - DOCKER_HOST=tcp://socket-proxy:2375
    secrets:
      - cf_email
      - cf_token

Interesting - you definitely shouldn't be seeing that. You can try to force by setting DEBUG_MODE=FALSE. I'm going to call in @mbjurstrom to review based on his recent PR with logging as I feel they are related.

I didn't really change anything about this as you always set debug as true when calling the cloudflare api
cf = CloudFlare.CloudFlare(debug=True token=token)
But I guess that wasn't shown until I changed it over to the logs.
https://github.com/tiredofit/docker-traefik-cloudflare-companion/pull/38
Should fix this. But please test it first. I don't have access to test it at the moment.

Found it: https://github.com/tiredofit/docker-traefik-cloudflare-companion/commit/bb3eeaea79dc56484cc373fdddacb4ed1569c4ac#diff-c978db2a7ba5bb07edd440eb15743f78f635035f5ec539ed0c1ab826beb0a0af
I had an if statement that was changing the value of the cf declaration to be added based on CONTAINER_LOG_LEVEL=DEBUG to insert that debug=True statement. I'll see if I can fix this.

Found it: bb3eeae#diff-c978db2a7ba5bb07edd440eb15743f78f635035f5ec539ed0c1ab826beb0a0af
I had an if statement that was changing the value of the cf declaration to be added based on CONTAINER_LOG_LEVEL=DEBUG to insert that debug=True statement. I'll see if I can fix this.

Oh I missed that line. But should be good now:)


Fix mono sqlite - attempt 2

It works, both Linux64 and Win64.  No #if tests.

This will need to be reviewed by Dave when he returns later in the week.

Ooh very nice! I'll review ASAP

The two biggies I can think of:

The Linux .so is version 3.8.6 while the Windows .dll and the .NET assembly are version 3.11.1.  Doesn't seem to bother it that the .so is a couple of minors behind: it seems likely that Halcyon isn't using the newer features.
The Windows .dll is now 64bit whereas it was 32bit before.  If SQLite was written correctly that shouldn't cause any issues when loading pre-existing state files. However if it wasn't, the difference in size_t, void*, &c. might be a problem.

Probably what needs to happen is the DLL needs to be rebuilt on the Windows side with the right options set so it makes both the portable managed DLL and the native SQlite.Interop.dll.  Then the Interop DLL needs to get put into the x86/x64 directory (or a .dll.config set up).  So on Windows it finds things.  On Linux I'd probably use the .so from the OS packaging along with a compiled sqlite3.dll.  Same trick for the Mac with it using the native dylib.

I specifically made it not use the interop process following https://gnuforensics.wordpress.com/2013/09/26/how-to-compile-sqlite-system-data-sqlite-on-linux-mono-using-xbuild/ - however I didn't test to see if it could be made to work with interop, so I guess that still might be an option.  However if there's a problem getting the 64bit sqlite3.dll to read state files that were created with a 32bit sqlite3.dll, going interop will cause even more issues - plus still requiring the segregation of linux (64bit only) and Windows 32bit region servers in separate grids.  Again, this is all predicated on IF there's a problem reading a 32bit-made sqlite file with a 64bit sqlite3.dll.
Another issue I ran across: it seems .NET/Mono doesn't know how to go looking for system libraries - nothing worked until I copied the .so from my system library folder to the Halcyon/bin.
A side question: do you have any 32bit region servers?  If not, then it's probably not a problem to just go 64bit.

Well the downside. as I understand it, is that not using Interop means all managed code.  So on Windows it wont be using the optimized Native binaries on Windows.   I may have that wrong, the documentation is  pretty sparse in this area.    Its great you got the managed approach to work though.  If thats good enough then it should ok as is.

Well, a proof would be to attempt to run this in Windows without the
sqlite3.dll.  It should fail.
What I did here was to compile on Linux using Mono an edition of
System.Data.SQLite.dll assembly that was designed to connect to a
non-interop native library in a way that worked not matter whether it was a
.so on Linux or a .dll on Windows. I did that following the above linked
GNUForensics instructions.
After that I found that it was complaining that the native library was
missing, so I cp'd from my Ubuntu 64bit system's installed sqlite, found
via the find command somewhere off in in a system 64bit lib folder, into
the Halcyon/bin folder.  I did this assuming that the .NET/Mono assembly
compiled above might be not looking at the system library folders.  Sure
enough I was right: it worked, so I added the .so and the changed assembly
to the repository and committed.
Next step was to test under Windows.  I pulled the changes from my repo,
compiled Halcyon, and found that it threw an exception when trying to
access the sqlite3.dll - wrong bitness.  Sure enough: the sqlite3.dll that
was being shipped with Halcyon was 32bit.  So I pulled the SQLite
amalgamation source and recompiled for 64bit - which was an adventure as
the build instructions on SQLite's site are wrong: follow the instructions
and your dll won't have any exported functions.  Thankfully I found a page
that gave the right instructions.  Got that in place and it worked.  (I'm
skipping over the big long fight trying to comprehend the
not-well-explained exceptions .NET was throwing so I could even find a
correct solution...)
On Tue, Oct 13, 2015 at 8:13 AM, mdickson<EMAIL_ADDRESS>wrote:

Well the downside. as I understand it, is that not using Interop means all
managed code. So on Windows it wont be using the optimized Native binaries
on Windows. I may have that wrong, the documentation is pretty sparse in
this area. Its great you got the managed approach to work though. If thats
good enough then it should ok as is.
—
Reply to this email directly or view it on GitHub
https://github.com/InWorldz/halcyon/pull/48#issuecomment-147745512.

I'll review this one tomorrow

I've confirmed that under windows the state saves are working, and indeed, when I renamed the native sqlite dll I got an exception as expected:
Exception thrown: 'System.DllNotFoundException' in System.Data.SQLite.dll
I'd have to conclude these changes are using the native sqlite on windows. This begs the question what our previous configuration was doing since the 32 bit DLL should not have loaded in the halcyon process which has 64 bit specified.
In either case, though this is a pretty big change, I'd like to move forward with it. As part of our first Halcyon test rollouht, we just need to make sure this DLL has no problems loading script states in our main grid tests starting with the beta grid and moving on to oceans and mainland.

A file I missed, but not critical: bin/sqlite3.dll.version.txt' with the contents <IP_ADDRESS>` - the idea was to be able to have a note specifying what version the dll was compiled against so that if we need a newer version it's obvious where that one came from.

Please add the file in a PR and I'll pull it in


fix typo

resise correction to resized

That's just how we spell it in Australia :D. I tend to use north american spelling in code (since it's annoying having functions named unexpectedly there), but my native dialect for prose.

Actually, no, you're right. It's size in both places, lol. Thanks for the spelling lesson ;).


Restructuring of Randomized Linear Solver Code

Usage Examples
Example 1: Specifying just the linear system

using RLinearAlgebra

A = rand(100,40); x = rand(40); b = A * x;
z = rsolve(A, b);

Example 2: Specifying the default solver with 500 iterations

using RLinearAlgebra
A = rand(100,40); x = rand(40); b = A * x;
solver = RLSSolver(500)
z = rsolve(solver, A, b)

Example 3: Supplying an initial condition

using RLinearAlgebra

A = rand(100,40); x = rand(40); b = A * x;
z = rand(40);

rsolve!(A, b, z)

# check z for the solution

Example 4: Supplying a solver with an initial condition

using RLinearAlgebra

A = rand(100,40); x = rand(40); b = A * x;
z = rand(40)
solver = RLSSolver(500)
rsolve!(solver, A, b, z)

# check z for the solution OR solver.x

Approach
An iterative linear solver (even for a single iteration) should be composed of several items:

An initialization
The core mathematical routine
A logging system
A stopping criterion

For a randomized iterative linear solver, these components are the same, except we can be more specific:

An initialization
The core mathematical routine: the randomized/deterministic projection method, and  a routine for solving the resulting subproblem
A logging system
A stopping criterion

The code is organized around the latter three components, given that the first component can be readily supplied with a simple argument as shown in the examples.

The randomized/deterministic projection methods are encapsulated in src/linear_samplers.jl and specific methods are implemented in a directory src/linear_samplers/
The routines for solving the resulting subproblems from the random operations are encapsulated in src/linear_solver_routines.jl and specific methods are implemented in a directory src/linear_solver_routines/
The logging systems are encapsulated in src/linear_solver_logs.jl and specific methods are implemented in a directory src/linear_solver_logs/
The stopping criteria are encapsulated in src/linear_solver_stops.jl and specific methods are implemented in a direction src/linear_solver_stop/

Following @dmaldona philosophy, all of the remaining three components come with defaults for novice user an can be highly customized for more experienced users.
Implementation Specifics
Owing to this approach, the core of the solver is very simple. The specifics are then implemented by the specific methods. Here is the code for rsolve! is as follows (see src/linear_rsolve.jl)

function rsolve!(solver::O where O<:RLSSolver, A, b, x::Vector{Float64})

    iter = 0
    log_update!(solver.log, solver.sampler, x, (), iter, A, b)

    while check_stop_criterion(solver.log, solver.stop) == false
        iter += 1
        samp = sample(solver.sampler, A, b, x, iter)
        rsubsolve!(solver.routine, x, samp, iter)
        log_update!(solver.log, solver.sampler, x, samp, iter, A, b)
    end

    solver.log.converged = true
    solver.x = x

    return nothing
end

There are several important functions

log_update! updates the solver.log depending on the type of log and the type of sampler being used. The details are implemented in src/linear_solver_logs.jl and the corresponding directory.
check_stop_criterion uses the solver.log and the solver.stop to specify a specific stopping criterion. The details are implemented in src/linear_solver_stops.jl and the corresponding directory.
sample uses the solver.sampler to specify a particular type of sampling/sketching/selecting scheme. The details are implemented in src/linear_samplers.jl and the corresponding directory.
rsubsolve! implements a particular subsolver for the resulting problem depending on the properties of solver.routine. The details are implemented in src/linear_solver_routines.jl and the corresponding directory.

While the design appears to be modular, the interactions between the logs, stopping criterion, sampler and subsolver routines can be tightly coupled by the types of each of these structures.
Short -term Objectives
Depending on whether the PR is approved, the short term goals are as follows:

Implement more sampling routines, including block sampling procedures
Implement more subproblem solving routines with a focus on block methods (e.g., precondition krylov methods such as blendenpik, approximate krylov method a la Laura Grigori, exact subproblem solvers, other inexact subproblem solvers)
Implement more realistic loggers for randomized linear solvers (current research work with N. Pritchard)
Implement more realistic stopping criteria based on the more realistic logs.
Implement more convenience constructors for constructing an RLSSolver
Manual pages (everything has a docstring)

@dmaldona I have made all changes requested.


TextBox control Language property is not working

Describe the bug
TextBox control Language property is not working.
Steps to reproduce the bug

Create a simple sample with TextBox
Set Langauge as "ar-SA"

Expected behavior
Arabic language should be applied.
Screenshots

Version Info

NuGet package version:

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

Additional context

duplicate,
https://github.com/microsoft/WindowsAppSDK/issues/1043

@ssaran046 This is the primary bug we're tracking since it's a WinUI bug (we transferred this bug to the WinUI repo).

@ssaran046 With this issue having moved repos it's missing a bit of the environment/setup info we'd expect for a WinUI bug. Can you fill in the table above with what version you're seeing this with? And when you say "is not working", do you mean it follows the wrong language? Or something more (for example there are linked bugs here that indicate text not showing up)?

@chrisglein I have seen this issue on ProjectReunion 0.8.1 and I saw the problem on May 2020 Update (19041).
"is not working" - what I mean is, there will be no effect when changing the Language property.

@chrisglein Chris Glein FTE I have seen this issue on ProjectReunion 0.8.1 and I saw the problem on May 2020 Update (19041).

Thanks for that clarification. That means this looks like a regression from UWP to WinUI3.


template_params argument

What does template_params do and how to use it? I tried to understand, but it looks that it's just redundant in terms of the main package.
Or is there some hidden magic I miss?

It was introduced in https://github.com/cweill/gotests/pull/90, for users who want to call gotests from a separate binary.

Sorry, I mean what this param does in main.go, as argument it does nothing. TemplateParams is quite clear down the road of the output package.

You're right, it looks like a bug where I forgot to pass it to the Options in #90. Would you like to send a PR to fix it?

You have already assigned this to me =), but I will put this in the queue as time isn't my best friend.


Fix MySQL 8.0.x incompatibilities

This patch accommodates some breaking changes introduced with MySQL 8.
Closes #1959
In a nutshell, the caching_sha2_password plugin (which is used by default since MySQL 8.0.4) hashes the password using SHA-256 and, after a first successful authentication attempt, saves it in a cache. That first attempt needs to be done under one of two conditions. The client either uses SSL and sends the password as clear text or it encrypts the password using the RSA public key generated by the server. On any subsequent attempt, until the password is somehow removed from the cache or the server shuts down, these rules no longer apply.
The handshake process remains unchanged when connecting to any server with version lower or equal to 8.0.3. Whereas for 8.0.4 or above, the process is now the following:

the client sends a ClientAuthenticationPacket with a scramble computed using a SHA-256 hash
if the password is not cached, the server sends back a PerformFullAuthenticationPacket
if the client uses SSL, the password is sent to the server (as clear text) via a ClearTextPasswordPacket to which the server replies with a OkPacket
otherwise it uses the server authentication public key compute the scramble, sending a AuthSwitchResponsePacket to which the server replies with a OkPacket
if the client does not know the server public key (is not provided by the user), it requests it from the server, which sends it back using a AuthMoreDataPacket
after a first successful authentication attempt, and until the password is cached, the server will reply to the initial ClientAuthenticationPacket with a FastAuthSuccessPacket (which basically just signals that an OkPacket will follow)

If the account is created using the mysql_native_password authentication plugin, the client will just fall back to the "traditional" process during the handshake, keeping compatibility, by default for any previously supported server version.
MySQL 8.0.2 disables the local_infile server variable by default, which breaks a couple of integration tests. The tests were updated to enable the feature by themselves (something that does not have any effect on older server versions and allows the tests to pass with newer versions).
Additionally, one of the integration tests was updated to avoid failing after the first run (using any server version) since it tried to create a table that already existed from the previous runs.

This is awesome, thank you soo much! The only real issues I'm seeing with the PR to continue to work on is the following:

[ ] Get support working on Node.js 0.10 and below, or we can decide if this PR should be shelved for the next major release as breaking change to drop those Node.js versions. Being a widely-used base driver we try to support as many Node.js and MySQL versions as possible at once to make it easy to use for everyone.
[ ] There may not be enough coverage of the new code added, as the coverage with the PR is decreasing. There are probably some areas that can use tests.
[ ] MySQL 8 image needs to be stood up on the CI server to test against. This is done in the .travis.yml file similar to our current MySQL / MariaDB matrix.

To address your points.
Regarding the first one, I've missed those tests (sorry), but as far as I can tell there is no sane way to use RSA public key encryption on node <=v0.12.0. There are some packages on npm that try to add support through native addons, but I think that might not be a good idea here, since it's pure JavaScript. So the only easy way to work around that is to require users on those older node versions to use SSL when connecting to new MySQL versions. If you are OK with that, we can go that route, otherwise, going in the next major version is probably the best possibility (I guess that's mostly your call).
Missed the coverage as well, I'll try to fix that ASAP.
Again, missed the CI config. Should be fine, at least on Travis. However, besides that, I suggest we also create a test account to run the tests (besides root) to make sure the handshake works fine for both empty and non-empty passwords, which lead to different behaviors. Any issue with that? Other than that, I guess on windows we are out of luck anyway since appveyor only seems to support MySQL 5.7, correct?

Regarding the first one, I've missed those tests (sorry), but as far as I can tell there is no sane way to use RSA public key encryption on node <=v0.12.0. There are some packages on npm that try to add support through native addons, but I think that might not be a good idea here, since it's pure JavaScript. So the only easy way to work around that is to require users on those older node versions to use SSL when connecting to new MySQL versions. If you are OK with that, we can go that route, otherwise, going in the next major version is probably the best possibility (I guess that's mostly your call).

Ah, very sad :( Did you do a through search on this (it sounds like it)? Because I can always help if you want to focus on the other parts, but just asking before I duplicate the search for no reason. If we really have to drop supported versions, we can, and it wouldn't particularly delay this, just wanted to make sure it was actually required to do so before saying we should :)

Again, missed the CI config. Should be fine, at least on Travis. However, besides that, I suggest we also create a test account to run the tests (besides root) to make sure the handshake works fine for both empty and non-empty passwords, which lead to different behaviors. Any issue with that? Other than that, I guess on windows we are out of luck anyway since appveyor only seems to support MySQL 5.7, correct?

Yea, so in Travis CI it just uses a Docker image. You can make multiple non-root accounts too; the current setup was just what was needed for testing, so if MySQL 8 requires more elaborate setup, that sounds fine to me. I would ignore AppVeyor.

I'm using crypto.publicEncrypt() which was added in v0.11.14, and it uses native bindings which I assume were also added around that time. There are some libraries on npm (e.g. ursa and dcrypt) that claim support for earlier node versions, however they seem to implement their own native bindings, which I guess would, sort of, defeat the purpose of this module (or maybe not, you tell me).
There's this node-rsa thingy which apparently is pure JS, but I couldn't make it work and it's pretty slow. I've also seen other approaches like delegating work to openssl via a child process, but that's probably not a good idea either, since it introduces a huge moving part. There's also this forge kit which brings the entire browser kitchen sink with it and I also haven't managed to figure out how it worked.
So, no good news from my side. However, if you can do your own research, that would, of course, be great. However, I'm not a cryptography expert, but from what I can tell, the reason I have not found anything good is because this RSA thingy in pure JavaScript is still not really feasible (number precision, primes and what not).
My suggestion here is to fail with a client error (the HANDSHAKE_SECURE_TRANSPORT_REQUIRED introduced by this PR) when process.env.version tells us that we are below v0.11.14, which basically would be the default behavior using { ssl: false, secureAuth: false }. It's a pretty naive solution but at least it's not really a breaking change, because existing implementations will keep working as expected with MySQL 5.7 and, let's be honest, people wo don't upgrade from those ancient node versions will, almost for sure, not upgrade MySQL to series 8 as well.
However, if you still feel you are better off bumping the major version, that's fine.

Sorry I didn't reply. Your comment on a new issue reminded me of this. Yea that works: if the server requires an auth that the client can't do we can error out. We should feature detect that, though, not sniff the version string.

Ok. Just to be sure. Are you talking about sniffing the Node.js version string, MySQL version string, or both? Because we have two different issues right now.
Supporting MySQL 8.0.4 (previous DMR release) which expects the password to be encrypted with the server public key using RSA_PKCS1_PADDING, whereas since 8.0.11 (the new GA release), the encryption should use RSA_PKCS1_OAEP_PADDING. We can always retry the handshake, but the only way to avoid that is by "sniffing" the MySQL server version.
Regarding the Node.js encryption APIs, I guess we can always resort to duck typing to check if they are available (or not) instead of sniffing the version. So, no problem here.
Let me know if there is something else that bothers you about these or other issues. I'll try to push this forward ASAP. Maybe during the weekend.

Why do you have to sniff the server version? Shouldn't the server say which one it expects directly? I'm just asking because I think version numbers break down when we start talking about compatible things like MariaDB, right? They may share some version numbers.
I have done Node.js version sniffing before and when it broke the Node.js collaborators just said I should never have been version sniffing their versions and told me feature detect only. So if that is the official direction from there I would go with that.

The case where we might need to sniff the server version is to distinguish MySQL 8.0.10 (pre GA, first one introduced chaching_sha1_passwors) from MySQL 8.0.11 (GA). since 8.0.11 was released yesterday one also could not support 8.0.10, as nobody should be using that RC anyways.
jojannes
On April 20, 2018 1:38:35 PM GMT+02:00, Douglas Wilson<EMAIL_ADDRESS>wrote:

Sorry I didn't reply. Your comment on a new issue reminded me of this.
Yea that works: if the server requires an auth that the client can't do
we can error out. We should feature detect that, though, not sniff the
version string.

I like the idea to either (a) just ignore the pre GA padding or (b) retry with the old padding because that would not involve the version sniffing :+1:

@dougwilson I've kept the option of providing a custom key padding, to allow users to exceptionally connect to a MySQL 8.0.4 RC server. In any case, the default value is always RSA_PKCS1_OAEP_PADDING which means it will work, by omission, with the latest 8.0.11 GA version.
I couldn't find a sane way to add an additional Travis CI test matrix for users with non-empty passwords though. I was constantly bumping into an authentication error (I believe it might be related to the mysql docker setup and environment variables mixup), but maybe you can give it a try.
Let me know if there is something else you would like to see in this PR. Otherwise, feel free to merge it! :wink:

This is awesome, thank you. I've been reviewing it, and sorry I didn't reply that I was in progress. I did find at least one regression just in existing behavior so far, and pushed a test to cover this to master so we have it covered for the future as well. I noticed the change for the ChangeUser sequence no longer passes any of the options down to the Sequence constructor, which differs not from every other sequence. Not sure what the purpose of that change was, but it at least prevents the timeout option from getting passed down.
I'll look at what the fix is and push it up to your PR branch here 👍

The merge process last so long, and the pr is still not merged. So it there any alternative way to solve the problem of "ER_NOT_SUPPORTED_AUTH_MODE: Client does not support authentication
authentication "? I am nodejs startup and just want to connect to mysql 8.0 for demo.

@fan123199 everything should still work if you use the mysql_native_password plugin.
ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password BY 'YourRootPassword';
-- or
CREATE USER 'foo'@'%' IDENTIFIED WITH mysql_native_password BY 'bar';

In addition to what @ruiquelhas said, I also had to:
mysql> FLUSH PRIVILEGES;
Query OK, 0 rows affected (0.00 sec)

And then restart the server.
If you are using multiple MySQL users, ensure that you update all of them. In my case I also had to use '%' instead of 'localhost'.

Any updates on this? Unable to authenticate with MySQL 8

@Osuriel

Is this a good fix that can be deploy to production or is this just a workaround for getting things working locally?

The new default authentication plugin is more secure because it's using SHA256 instead of SHA1 to hash the password, but it's not available for MySQL versions below 8.0, which means most existing MySQL production instances are not using it as well. So, it all boils down to how much you can trust SHA1 (for which collision attacks have already been performed in 2017). I guess having a good password should help to mitigate or even prevent that.

I am also assuming that what is happened is that 8.0.11 MySQL Community Server is just not compatible with the npm mysql library right out the box at the moment. To make it compatible right out the box You have submitted this bug fix but it hasn't been merged to master yet, and that is the reason it still doesn't work for me right out the box even tho i have the latest version of the mysql dependency.
Is my assumption correct?

Correct.

If I am correct about my assumption, is there a practical way to implement the bug fix to my codebase before it is merged into master? and if there is, is that something that would be advised, or good practice?

I don't know if it is a good practice or not, it mostly depends on your internal policies and processes (and how much you would be "trusting" the fork in this case), but you can install and link npm packages from a git repo even using a specific branch or commit.
$ npm install ruiquelhas/mysql#feature/caching-sha2-password

Any update? Still unable to connecto to MySQL 8

@ruiquelhas thank you so much! You just helped me become a better developer hannibwas yesterday! 💪🏽🔥

@dougwilson Is this still planned to be merged? It obviously keeps accumulating conflicts while it is not :(

This PR regresses behavior and cannot be merged until fixed as I noted above. I volunteered to fix the PR but have not gotten to it. Anyone is welcome to fix the regression in the PR, though, and will merge.

@dougwilson Thanks for the update! Not sure I'll get to it in the nearby future personally, though (but it's a possibility).

It's np. I did get it mostly rebased and fixed locally but didn't quite finish. I'll bring this close to the top of my current todo list.

I'll look at what the fix is and push it up to your PR branch here

@dougwilson I was under the impression you would take over from there, and just assumed something else was blocking this. In any case, I'll gladly help to push this forward. Please, let me know what I can do.

Alright, perfect. Let me know if there is something I can help with.

I think you should keep track of this issue :) still on 8.0.

Hi all,
Thanks for your effort!
But please merge this PR. It's been around 8 months that we have this compatibility issue with mysql 8.
Thanks 🎉

Thanks to all who have worked hard on the caching_sha2_password support. Are there plans to merge this soon?

Hi all,
Thanks for your effort!
But please merge this PR. It's been around 8 months that we have this compatibility issue with mysql 8.
Thanks 🎉

I  don't get what is the wrong with this comment. It was pretty gentle. But It seems, you spend all the time for marking comments as something, not try to solve this BIG ISSUE which isn't solved for almost 10 months!


Added windows_function_app and linux_function_app

Added windows_function_app and linux_function_app as function_app will be deprecated.
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/function_app

Great PR but needs top solve conflicts

Can this be updated for the windows/linux app services?


[Servers] Replace ERR_FAIL_COND with ERR_FAIL_NULL where applicable

Just opening the last one to make it easier to track conflicts

Putting in a few I found recently

Thanks!

Thank you!


Question

Is there any options to convert sentry hash file to bin file avoiding 7 day trade ban?

By the nature of hashes, you can't convert a hash into the original content. The setSentry() method will accept both the full sentry and the hash.
Please direct future questions to the Steam group.


Preferences in Message Center missing

I don't know if the guide is wrong/outdated or if it's a permission thing, but me and my colleague have access to the message center but the preferences button is missing. Can you update the guide to reflect this behavior?

[Enter feedback here]

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

ID: 29f3771a-247c-4f85-992a-e671285c70bd
Version Independent ID: f3c2890a-2470-9349-ff77-924cf8566365
Content: Message center - Microsoft 365 admin
Content Source: microsoft-365/admin/manage/message-center.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@aktifit Thank you for your feedback. I have tested in multiple tenants across different regions and I'm able to view the "Preferences" option.

Not sure, if it was a glitch while you submitted this feedback. If you continue to not see the preferences option, I'd recommend you to contact Microsoft support via the ned help link on the admin center.
Hope this helps!
Thanks
Sri

@msbemba
It seems there is an issue when accessing the Service Health page as a guest user, I can see that it appears if I log in as a regular admin that is homed in the same tenant. I'll open a support ticket, thanks for the hint though.


Fails when upgrading to 3.7.0 with an error message about extends

Bug Report

TSLint version: 3.7.0
TypeScript version: 1.8.9
Running TSLint via: CLI

Running
We run tslint from make file. Everything works fine with any tslint previous to 3.7.0.
tslint -c tslint.json `find src/ts -name *.ts | xargs`

I have not added anything related to the new extends feature, so my tslint.json only contains a big rules block.
Actual behavior
tslint -c tslint.json find src/ts -name *.ts | xargs
/Users/vertti/.npm-packages/lib/node_modules/tslint/lib/configuration.js:112
throw new Error(("Invalid "extends" configuration value - could not require "" + relativeFilePath + "". ") +
^
Error: Invalid "extends" configuration value - could not require "tslint.json". Review the Node lookup algorithm (https://nodejs.org/api/modules.html#modules_all_together) for the approximate method TSLint uses to find the referenced configuration file.
Expected behavior
I'm expecting normal linting.

I can confirm this from my side too. I tried using npm@3 and npm@2, the bug happens with both. Downgrading TSLint to 3.6.0 worked for me.

Sounds like a bug :cry:. I'll look into it sometime in the next few hours!

@vertti @staltz So this is actually the intended behavior, sort of. What happened is we introduced a way so that you could load configurations that were in an NPM package. This was mainly for the new and optional extends field of tslint.json files. The semantics are similar to the semantics when using require in Node:

An absolute path loads a file from the location it points to.
A relative path starting with ./ or ../ loads a file relatively.
A plain string value, such as "tslint-config", tries too load the main file of an npm package called "tslint-config".

What's happening here is that the -c CLI option uses the same code and logic to load config files, and thus in your cases is trying to load an npm module named "tslint.json". You can resolve this bug on your end by specifying the path as "./tslint.json".
Nonetheless, this clearly is unexpected behavior to users. I see two solutions:

Better education/error messages. I made a mistake when writing this error message - I was thinking it would only get thrown in the context of using extends, but clearly that is not the case. With good documentation / clear error messages, users could probably figure out to start their path with a ./.
Change it so that the -c option always treats its argument as a path and never attempts to resolve it as a module. Module resolution would still be available as is for the extends option.

Shouldn't this be a breaking change version update then?

agree with @staltz that it's not the expected, that an upgrade breaks my working version in perspective of backwards compatibility. but if you don't want to make a breaking change version update i'd like your idea of improving the error message, so i know as a user what i have to change in my configuration to make it work again.

TSLint 3.7.0 has only been published for less than a day, so I'm happy to revert this semi-unintended breaking change (via option 2 described above). I don't think this will mess up anyone's workflow so shouldn't be a problem.
I was more worried about the -c option having different semantics than the extends field, but since one is a CLI option and one is a field in a configuration file, I think slightly different semantics should be okay?

Filed #1084 to fix this

This should be fixed with version 3.7.1 which I just published. If you all are still running into any problems, let me know!

I'm still having issues with 3.7.1, see https://github.com/Microsoft/vscode-tslint/issues/41

Glad you found a work-around @timbru31. This is our fault and not the fault of vscode-tslint, https://github.com/palantir/tslint/pull/1093 should fix this though so that it will work correctly with or without the "./"

@timbru31 Fixed version is published (v3.7.2). Let me know if you run into any further problems


Custom Labels for Services and Pods

Discussed in https://github.com/artemiscloud/activemq-artemis-operator/discussions/480

Originally posted by jairogutierrezpayu January 30, 2023
Hi, for some policies i need add customized labels to some components generated dinamically such like services, wconsj even control-manager.
Is possible add customized labels to this components
In my case add labels to resources is requirement of FinOps and Security team for all resources in cluster.
Thx

@brusdev We have the same issue - we need a specific label applied to all resources for our project (enforced with a webhook that is owned by the cluster admins).
What are your thoughts on what the API should look like?  Would you add additional labels under Spec.DeploymentPlan.Labels?  Ex: Spec.DeploymentPlan.Labels.Routes, Spec.DeploymentPlan.Labels.Services, ...).
Or, create a new label element for each resource.  ex: Spec.DeploymentPlan.RouteLabels, Spec.DeploymentPlan.ServiceLabels, ...)
Or, create a new element - something like Spec.DeploymentPlan.GlobalLabels, and apply those to everything?
Or, just apply the value that's in Spec.DeploymentPlan.Labels to all resources that are created?
The last one seems the simplest to start with, so I'm going to start implementing that.  Changing it later would likely be simpler once the code for including the label is in place.

@davidlanouette Spec.DeploymentPlan.Labels sounds good to me too. @gtully @howardgao do you have any concerns?

That can work, the namer label builder can add the Spec.DeploymentPlan.Labels to its per CR list. One issue is that it would cause all resources to be modified/updated, on an upgrade, but I think we can freely modify labels with no impact. The labelBuilder labels are already applied to all resources.
currently Spec.DeploymentPlan.Labels is only applied to the stateful set template, along with the namer labels

There is also the issue #588 related to custom labels. The solution discussed in the issue #588 is that the ActiveMQArtemis CR should have the field labels in acceptor, connector and console too.


Node.duplicate() runs about twice as slow since node property duplication fix

Tested versions

Reproducible in: 4.3.dev5
Not reproducible in: 4.2.1.stable

System information
Godot v4.3.dev (b091162a9) - Fedora Linux 39 (KDE Plasma) - X11 - GLES3 (Compatibility) - NVIDIA GeForce RTX 4090 (nvidia; 545.29.06) - 13th Gen Intel(R) Core(TM) i9-13900K (32 Threads)
Issue description
Node.duplicate() runs about twice as slow since node property duplication fix, even on a plain Node with no properties or metadata set.
Before the fix
Iteration 0: 48.979 MB
Iteration 1: 48.984 MB
Iteration 2: 48.994 MB
Iteration 3: 49.014 MB
Iteration 4: 49.054 MB
Iteration 5: 49.132 MB
Iteration 6: 49.290 MB
Iteration 7: 49.606 MB
Iteration 8: 50.266 MB
Iteration 9: 51.588 MB
Iteration 10: 54.099 MB
Iteration 11: 59.252 MB
Iteration 12: 69.558 MB
Iteration 13: 90.169 MB
Iteration 14: 131.391 MB
Iteration 15: 213.836 MB
Done running in 3571 milliseconds.

After the fix
Iteration 0: 40.596 MB
Iteration 1: 40.601 MB
Iteration 2: 40.611 MB
Iteration 3: 40.631 MB
Iteration 4: 40.671 MB
Iteration 5: 40.749 MB
Iteration 6: 40.907 MB
Iteration 7: 41.221 MB
Iteration 8: 41.881 MB
Iteration 9: 43.203 MB
Iteration 10: 45.714 MB
Iteration 11: 50.867 MB
Iteration 12: 61.173 MB
Iteration 13: 81.784 MB
Iteration 14: 123.006 MB
Iteration 15: 205.451 MB
Done running in 5207 milliseconds.

I bisected the regression to 9851c1bdd893a6194fdbf1820d2fa07449d4f9fb (https://github.com/godotengine/godot/pull/87387).
cc @warriormaster12
Steps to reproduce

Use Node.duplicate() thousands of times in a project or more in a quick succession. The performance difference is noticeable in both debug and release builds.

Minimal reproduction project (MRP)
test_alloc_duplicate.zip

The pain never ends.

My first guess is that it is due to the function _duplicate_properties_node that I wrote. I'll be testing that theory in a moment.

Tested on M3 Pro Macbook Pro
Iteration 0: 58.707 MB
Iteration 1: 58.712 MB
Iteration 2: 58.722 MB
Iteration 3: 58.742 MB
Iteration 4: 58.782 MB
Iteration 5: 58.861 MB
Iteration 6: 59.018 MB
Iteration 7: 59.331 MB
Iteration 8: 59.992 MB
Iteration 9: 61.313 MB
Iteration 10: 63.824 MB
Iteration 11: 68.978 MB
Iteration 12: 79.284 MB
Iteration 13: 99.895 MB
Iteration 14: 141.118 MB
Iteration 15: 223.563 MB
Done running in 2246 milliseconds.

After commenting _duplicate_properties_node
Iteration 0: 58.383 MB
Iteration 1: 58.388 MB
Iteration 2: 58.398 MB
Iteration 3: 58.418 MB
Iteration 4: 58.458 MB
Iteration 5: 58.537 MB
Iteration 6: 58.694 MB
Iteration 7: 59.008 MB
Iteration 8: 59.668 MB
Iteration 9: 60.989 MB
Iteration 10: 63.501 MB
Iteration 11: 68.654 MB
Iteration 12: 78.960 MB
Iteration 13: 99.571 MB
Iteration 14: 140.794 MB
Iteration 15: 223.239 MB
Done running in 1596 milliseconds.

So my theory was correct.


doc: optimize ovs

Thanks for contributing!

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #3402
Special notes for your reviewer:

/hold


Index MC config metadata only when hypershift is detected

Description
I think we can move this code to the hypershift() function

cc: @mukrishn


(WIP) vcv-rack: 1.1.4 -> 1.1.5

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[x] 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"
[ ] 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
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Closed in favour of #70146


How to disable wrap mode is not clear from the help

The help says:
  -w, --wrap                       wrap mode (default true)

but I had trouble finding the way to disable wrap mode.
# NG
$ ov --wrap false sample.txt

# OK
$ ov --wrap=false sample.txt

If the help was something like this, it would be kinder:
  -w, --wrap=[true|false]                      wrap mode (default true)

Thank you for your issue.
This is confusing in how Cobra works and could be improved on Cobra's side. https://github.com/spf13/cobra/issues/1606
I didn't want to rewrite the help in a big way, so I decided to do it with a small fix.
It will be as follows.
  -w, --wrap[=true|=false]         wrap mode (default true)

I used SetUsageFunc to make it a customizable way because the previous PR had issues.


Update showcase.json

Add DooCoins
Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

@jakepeg Can you please do a rebase?


rever: disable single file support

EFM does support NULL root directories.
This reverts commit 0d2722a63e7e79bfc97ea10322ecbbfd48154e77.

Has the new release been cut? Also I'd rather hold off until this hits package managers.

Yes: https://github.com/mattn/efm-langserver/releases/tag/v0.0.38
I'm fine with holding this off (users can simply change their config)
Though I've fixed it in Debian already ;).

Would also be good to verify that the internal EFM root directory it infers with the root pattern setting that can be sent via workspace/didChangeConfiguraiton still works well.

How would you do that? (It seems fine when looking at the sources)

@mattn Sorry for the ping. Would you mind commenting on EFM's expectation?
I added support for launching EFM in the parent directory of the open file, without sending rootDirectory or workspaceFolders during initialization (sending NULL) if we can't detect the current file is a child of a git repository.
I see there is also root-markers in the EFM configuration file. I'm wondering if sending NULL disables any features/how EFM handles locating root-markers
Thank you!

I went ahead and directly committed this to master with a note about backwards compatibility.


[Requirement] Mapping of Qwerty/Transliterate Keyboards made for any generic Keyboard

w3 Reference : https://www.w3.org/TR/uievents-code/#key-alphanumeric-writing-system
https://hacks.mozilla.org/2017/03/internationalize-your-keyboard-controls/
Javascript event : event.data.domEvent["$"].code

Code Update : https://github.com/Vyshantha/multiscripteditor/blob/b93bf2808ef9577caedcd8b93d16df8e3a92568e/editorClient/src/app/rich-text-editor/rich-text-editor.component.ts


Ignore the final test on anagram (initially, only the first test should run)

Ignore the final test on anagram (initially, only the first test should run)

Thanks! :+1:


No Source Available in Test Explorer

Description
In the Unit Test Explorer I can see and run a unit test, but when I select the test it displays on the bottom, "No source available"
Steps to reproduce
I created a unit test project, and converted it to use the new project file format with PackageReferences. I added two Nuget packages, MSTest.TestAdapter and MSTest.TestFramework.
Expected behavior
I should be able to double click a test to navigate to the code
Actual behavior
I see a message "No Source Available"
Environment
Visual Studio v15.3.5
MSTest.TestAdapter v1.1.18
MSTest.TestFramework v1.1.18
I may not have properly converted the project to use the new project format. Is there a sample of a Unit Test project which uses PackageReferences (as opposed to packages.config?)

@jackbond: I guess the conversion went well because you are able to discover and run tests. There might be something else at play here. Could you turn on TestExplorer diagnostics from Tools -> Options -> Test -> Logging -> Logging Level? Set that to diagnostic and share the logs that show up in the Output->Tests pane with us please.

repro.zip
How about a solution that reproduces the issue?

I've resolved the issue by adding the Microsoft.NET.Test.Sdk Nuget. I skipped adding it originally because its Nuget description reads, "The MSbuild targets and properties for building the .Net core test projects." Perhaps that description could be updated?

@jackbond: Thanks for the repro. This is a new project system based project where the pdbs generated are portable by default. That explains why the Source Information logic fails since it currently only works on full pdbs.
Adding Microsoft.Net.Test.Sdk package brought in the following
<DebugType>Full</DebugType>

ensuring that the scenario works. You could go ahead and keep that package installed(which is added in by default for Unit test templates in the project system).
/cc:  @Faizan2304, @smadala for the package description change.

@AbhitejJohn
Thanks for the info. I converted a few more of our unit test projects, and one of them had an AssemblyInitialize attribute. When I ran a test, I got the error below. I haven't been able to recreate the problem in the repro project I sent you, any possible ideas? Thanks.
Method OurClassName.AssemblyInit has wrong signature.  Parameter 1 should be of type Microsoft.VisualStudio.TestTools.UnitTesting.TestContext.

@jackbond: Do you have a testsettings file specified for the run? If yes, can you try removing it or replacing with a runsettings instead?

Please reactivate if this is still blocking you.

Is there an issue tracking the fact that the test explorer doesn't work with embedded PDBs?

@johncrim From VS 15.5 test explorer should support it, Source-Information-For-Discovered should solve the embedded PDBs issue.

Thanks @smadala . I've found that VS 15.5 works with portable symbols, but not embedded. The linked document doesn't provide any additional info on support for portable vs embedded symbols.

@johncrim
From doc:

Use of better performant Roslyn APIs instead of reflection based approach on PDBs.

Indicates source info determined by Roslyn APIs, It shouldn't be matter what type the PDB is.
Debug type embedded working for me on 15.5.1. See the below image.

Just FYI, I had this problem with a .NET Console App targeting full framework, referenced from unit tests. Going to settings and setting Build -> Advanced -> Debugging information to Full fixed the problem.

Ok ,  I had this issue, but figured it out.    (visual 2022 specflow 3.9.74)
I had excluded the .cs specflow generated test files from the solution since they are recreated when project is built.
The tests run even if the .cs are not included on the solution, but the test explorer only sees the source file only if  included.


Docker Hub build broken

Hi! I think your repo restructuring lead to the Docker Hub build being broken: https://hub.docker.com/r/geoceg/arcgis-server/builds/ba3xjqz9cyojfpklpddxcgm/
I guess you must adjust the path to the Dockerfile in the build configuration on Hub.
Unfortunately, I get "Software authorization failed. Please make sure your authorization file is valid and try again.", which is not your problem :-)
Thanks for a very well documented project!

I removed the broken hub.docker.com entry and will have to create a new entry when I figure out how to have Docker Hub build the 4 containers this project requires from my one github repo.

It is possible to specify multiple sections within a Dockerfile to build
multiple containers. You can use the FROM command to start off new sections
within the same file. (Note however that if you supply "-t"  command line
flag to apply a tag while building, it will only apply to the final section
in the file.)
On Mon, Aug 28, 2017 at 4:10 PM Brian H Wilson<EMAIL_ADDRESS>wrote:

I removed the broken hub.docker.com entry and will have to create a new
entry when I figure out how to have Docker Hub build the 4 containers this
project requires from my one github repo.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Geo-CEG/docker-arcgis-enterprise/issues/1#issuecomment-325508368,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APMHt-jxM-rfRH2c3TwmO7Ydd45JUubdks5sc0j1gaJpZM4O7fqi
.

--
[image: photo]
Patrick Salsbury
Senior DevOps Engineer, Nebulaworks
408-499-3907 <javascript:void(0);> |<EMAIL_ADDRESS>| 5151
California Ave., #100 Irvine CA 92617
https://www.linkedin.com/company/nebulaworks?utm_source=WiseStamp&utm_medium=email&utm_term=&utm_content=&utm_campaign=signature
https://plus.google.com/+Nebulaworks?utm_source=WiseStamp&utm_medium=email&utm_term=&utm_content=&utm_campaign=signature
https://www.twitter.com/nebulaworks?utm_source=WiseStamp&utm_medium=email&utm_term=&utm_content=&utm_campaign=signature
https://www.facebook.com/nebulaworksco?utm_source=WiseStamp&utm_medium=email&utm_term=&utm_content=&utm_campaign=signature
https://www.nebulaworks.com?utm_source=WiseStamp&utm_medium=email&utm_term&utm_content&utm_campaign=signature


Fix wrong value returned by murmurhash64a/murmurhash64b on Windows

The functions murmurhash64a and murmurhash64b were returning wrong hex value when third parameter is set to true (meaning return type is hex string). In this case, sprintf was used with "%lx" as format although the input is a 64-bit integer value.
The fix is to use RING_UNSIGNEDLONGLONG_FORMAT for sprintf.
The program below reproduce the issue on windows: it should display "b4111e8f925d5920" but it displays "925d5920"

load "murmurhashlib.ring"

key = "Ring Language"

s = murmurhash64a (key, 1, 1)

? s

Thanks for the update

Thanks for the update


Unsized constant width

Consider this test:
module top;
    logic [63:0] s1c;
    assign s1c = '1 << 8;
endmodule

According to the SystemVerilog standard, '1 would mean setting all bits to '1, and then there's the shift of 8 bits.
Surelog reports '1 << 8 as a constant:
\_constant:
  |vpiDecompile:256
  |vpiSize:64
  |UINT:256
  |vpiConstType:9

I think this is incorrect, I would expect the result to be equivalent to:
64'b<PHONE_NUMBER>111111111111111111111111111111111111111111111100000000.

Fixed by #3696


IHostedService in generic host can inject scoped dependencies, while IHostedService in webhost cannot

Describe the bug
IHostedServic implementation in generic host can inject scoped dependencies, while IHostedService in webhost crashes.
To Reproduce
.net core 2.2
Inject a scoped dependency in a ihostedservice implementation in a generic host, the service starts and the dependency is available for use.
            var host = new HostBuilder()
                .ConfigureServices((hostContext, services) =>
                {
                    services.Configure<HostOptions>(option =>
                    {
                        option.ShutdownTimeout = System.TimeSpan.FromSeconds(20);
                    });

                    services.AddScoped(typeof(ScopedDependency));
                    services.AddHostedService<Service>();
                })
                .Build();

            host.Run();

Inject a scoped dependency in a ihostedservice implementation in a webhost, the host crashes on startup.
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddMvc().SetCompatibilityVersion(CompatibilityVersion.Version_2_2);

            services.AddScoped(typeof(ScopedDependency));
            services.AddHostedService<Service>();
        }

Expected behavior
I would expect the IHostedService under a webhost to function the same way as in a generic host.

A singleton can't depend on a scoped service, that leads to horrible leaks. The reason it "works" in the generic host case is because WebHost.CreateDefaultBuilder sets this up https://github.com/aspnet/AspNetCore/blob/85f1f3d893d3c2b776cd81504f1b07282a86c163/src/DefaultBuilder/src/WebHost.cs#L191-L194.
We've just added this similar functionality in 3.0:
https://github.com/aspnet/Extensions/blob/67d565dedbb9b8c1ef85391aaf92a3498d066293/src/Hosting/Hosting/src/Host.cs#L60-L63
So it'll fail just as magically in 3.0 in both cases.


Improved worker + watcher management

The watcher process already has a commnunication channel to each worker, the IPC socket. This socket is currently pretty dumb.
I think this could be easily extended to share more information between worker + watcher.
Examples:

Measure number of executed requests
Measure worker utilization (https://github.com/amphp/amp/issues/161)
Detect unresponsive workers (too long running requests, dead locks, whatever)

The IPC protocol could be command based and bidirectional. E.g.
4 byte cmd length:cmd string:4 byte data length:data

Commands from watcher to worker could be

stop
heartbeat (passive)
stats

Commands from worker to watcher could be

log
heartbeat (active)

Another option would be to use a HTTP2 connection between worker and watcher. That would be even more consistent with the Aerys architecture.
Having all that a HTTP based management interface for the watcher would make sense to issue commands to the watcher like:

Restart
Gather stats (Utilization, Number of workers, ...)
Change worker count
...

Why HTTP management?

Aerys is HTTP
Distributed management is easier with HTTP than with SSH/CLI
Reporting API makes sense as a RESTful service

I think most of these don't have to go to the master to be submitted to some time series database. You could send the metrics for each worker directly to the TSD.
Completely unresponsive worker detection should probably be built-in directly.
Anyway, the IPC communication probably needs a redesign.

@brstgt We have built a HTTP-independent solution in https://github.com/amphp/cluster. Could you review the API and share your thoughts? Should we allow completely user-defined IPC? Is it enough to have built-in logging and heartbeats?

Worker management is no longer part of amphp/http-server, thus I'm closing this issue. Issues should instead be opened for amphp/cluster.


List of URLs to manage firewall settings

The list of domains mentioned at https://learn.microsoft.com/en-us/azure/security/fundamentals/azure-ca-details#certificate-downloads-and-revocation-lists doesnt seem to match the list mentioned in the last point of https://learn.microsoft.com/en-us/azure/security/fundamentals/tls-certificate-changes#will-this-change-affect-me. This can create confusion for customers who are managing their firewalls/proxies to limit/allow specific URLs.

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

ID: 61c4adec-3601-5d22-19f5-78fb13189d3c
Version Independent ID: c52fb2be-28d1-a0cf-304a-54ceaa12797f
Content: Azure Certificate Authority details
Content Source: articles/security/fundamentals/azure-CA-details.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @shlipsey3
Microsoft Alias: sarahlipsey

@imtiazh
Thanks for your feedback! We will investigate and update as appropriate.

Hello - thanks for the feedback! I've reached out to the product group to get clarification on the URLS. The CA Details article is the current source, but I'm confirming the status of the three that are in the TLS article, but not the CA details article.

Hello! I've confirmed with the product group that there were three URLs listed in the TLS article that were no longer used. I've updated the list and provided a link to the complete list on the Azure CA details article. While the TLS article is marked as no longer being updated, it made sense to remove outdated information.
The updates should be live later this morning.
Thanks again for your feedback!

#please-close


How to stop using geolocation on iOS?

Hello. I'm using latest stable version from nuget, and I can't find, how to stop using geolocation, when my doesn't need it anymore?
I'm using StartListeningAsync to start usage of geolocation. When I start using geolocation, there is an arrow appearing in the top of iOS device.
When I don't need it anymore, I call StopListeningAsync, the geolocation arrow is still showing, that means, that my app is still using geolocation services.

@overzealus When you call the StopListeningAsync the app should should tracking gps. Could you provide following details? It will help us to troubleshoot the issue
Bug Information
Version Number of Plugin:
Device Tested On:
Simulator Tested On:
Version of VS:
Version of Xamarin:
Versions of other things you are using:
Steps to reproduce the Behavior
Code snippet

I have noticed that iOS takes about 15-30 seconds to stop showing the icon.


docs: add description to "remote_path" in "gas cp" in cli_commands.rst

Pull Request Test Coverage Report for Build 970650758

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.091%

Totals

Change from base Build 968000409:
0.0%

Covered Lines:
4914

Relevant Lines:
5775

💛  - Coveralls


require multiple route

hello
$app = new Slim\App($setting);

require __DIR__ . '/routes/route.php';

require __DIR__ . '/routes/login.php';

$app->run();

route.php
$app->get('/', function (Request $request, Response $response, $args) {
    return $this->view->render($response, 'auth.twig');
})->setName('home');

login.php
$app->get('/logout', function (Request $request, Response $response, $args) {
    destroySession();
    return $this->view->render($response, 'auth.twig');
})->setName('logout');

i don't know 500 status
thank you

Can you ask the question on http://discourse.slimframework.com/ and can you also make the question a bit more clear, I have no idea what you are asking at this point.


Debezium server is failing to create a table because of an invalid table identifier

Hello,
I'm reaching out to discuss an issue we're facing while setting up the Debezium-Iceberg integration in our architecture. We followed the steps outlined in Chris Cranford (Naros)'s blog post (https://debezium.io/blog/2022/09/30/debezium-oracle-series-part-1/) and ensured that the Oracle database user has been granted the necessary privileges as described in the Debezium documentation.
However, we've encountered an error that we're struggling to resolve. The error message in the logs indicates that the Debezium server is failing to create a table due to an invalid table identifier: apacheiceberg.debeziumcdc_aws_DNT_TEST. This issue arises during the processing of change data capture (CDC) events using the Iceberg sink connector.
Attached is a screenshot showing the exact error message for reference.
We would greatly appreciate any guidance or assistance you can provide to help us resolve this issue.
Thank you for your support.
Best regards,
Dhruv Mevada

@mevadadhruv as mentioned before please check that Glue catalog naming rules are supporting the naming you are using.
please test with Skip Name Validation config
https://github.com/memiiso/debezium-server-iceberg/issues/365#issuecomment-2181033713
the details of glue catalog here https://iceberg.apache.org/docs/nightly/aws/?h=glue#glue-catalog

also i was checking this issue in the debezium community side where Vojtech Juranek suggested us the "SMT". from the iceberg side it'll also help right ?

Hi @ismailsimsek , we are successfully able to get data to a3 bucket through debezium and iceberg.
When we create a table in lower case its work but for uppercase its showing errorand unable to create them.
Please let me know how should I bypass this thing or need to add any parameter in application.properties
please share the details or examples if possible.
Regards,
Govinda

you could add this dependency to your project and use this SMT https://github.com/jcustenborder/kafka-connect-transform-common
examples how to use it are available int the project.
additional details https://stackoverflow.com/a/60860628/2679740

hi @ismailsimsek,
Thanks for giving quick reply everytime, and also thanks for this helps now the thing is recent solution for our error SMT it's not feasible as we are hoping because the link which you share it totally related to kafka, which we are not using, we are using  debezium iceberg consumer from github repository without kafka. now let me share our whole things like root level pom.xml file, application.properties file, error logs in file so you can guide easily. also after checking your solution we have also add this jar files
connect-utils-0.7.173.jar, freemarker-2.3.31.jar, kafka-connect-transform-common-<IP_ADDRESS>.jar, ws-commons-util-1.0.1.jar , xml-apis-1.4.01.jar, connect-utils-jackson-0.7.173.jar, junit-3.8.1.jar, value-2.8.2.jar, xercesImpl-2.12.1.jar.
application_properties.txt
errorlog.txt
pom_xml.txt

hi @ismailsimsek ,
Please, do let me know if you are finding difficulties to understand this !
Thank you,
Dhruv

Good Morning, @ismailsimsek

@ismailsimsek ?

this option is now implemented in 0.6.0.Final release

Hi @ismailsimsek,
Thank you so much for addressing our issue and providing a release that resolved the problem. We really appreciate your support.
Currently, we are able to retrieve data from our source, but it's only working for a single table. After reviewing the Debezium documentation, we understand that Debezium should be capable of capturing data from multiple tables simultaneously.
However, we're now encountering an issue where we can't achieve this. We've attempted several different scenarios to resolve the problem, but unfortunately, we haven't been successful.
Could you please assist us with this? Below, I'm sharing our application.properties file and the logs we're receiving for your reference.
Thank you in advance for your help!
debe-logs.txt
app_pro.txt


ip route add can fail

In particular, it can fail if the route exists and is being imported from BIRD. We should use "ip route replace" instead.

Fixed in PR #181.

Tested fix. Works fine.


[vcpkg] CMAKE_C_COMPILER and CMAKE_CXX_COMPILER are not set

In vcpkg-configure-cmake, we doesn't set CMAKE_C_COMPILER or CMAKE_CXX_COMPILER, causes cmake to automatically acquire the compiler.
After llvm built clang++, when we set up and used ninja or Visual Studio, cmake obtained and detected clang++, which caused some parameters (such as /nologo /DWIN32 /D_WINDOWS) we set to destroy the detection process.
Example error:
CMake Error at D:/downloads/tools/cmake-3.18.2-windows/cmake-3.18.2-win32-x86/share/cmake-3.18/Modules/CMakeTestCCompiler.cmake:66 (message):
  The C compiler

    "D:/installed/x64-windows/bin/clang.exe"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: D:/buildtrees/ignition-msgs1/x64-windows-rel/CMakeFiles/CMakeTmp

    Run Build Command(s):D:/downloads/tools/ninja/1.10.0-windows/ninja.exe cmTC_f8bc8 && [1/2] Building C object CMakeFiles/cmTC_f8bc8.dir/testCCompiler.c.obj
    FAILED: CMakeFiles/cmTC_f8bc8.dir/testCCompiler.c.obj
    D:\installed\x64-windows\bin\clang.exe   /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /MP   -g -Xclang -gcodeview -O0 -D_DEBUG -D_DLL -D_MT -Xclang --dependent-lib=msvcrtd -MD -MT CMakeFiles/cmTC_f8bc8.dir/testCCompiler.c.obj -MF CMakeFiles\cmTC_f8bc8.dir\testCCompiler.c.obj.d -o CMakeFiles/cmTC_f8bc8.dir/testCCompiler.c.obj -c testCCompiler.c
    clang: error: no such file or directory: '/nologo'
    clang: error: no such file or directory: '/DWIN32'
    clang: error: no such file or directory: '/D_WINDOWS'
    clang: error: no such file or directory: '/W3'
    clang: error: no such file or directory: '/utf-8'
    clang: error: no such file or directory: '/MP'
    ninja: build stopped: subcommand failed.

default set CC=cl CXX=cl ?

Mostly, clang should go to tools and not bin folder?

Yeah, we shouldn't ever be finding a compiler in the "installed" directory. @Voskrese 's suggestion is probably the right fix.

@cenit That also seems appropriate but I suspect it might require a more.... invasive modification of llvm's build than we would like. If that's possible it would be great!

default set CC=cl CXX=cl ?

That breaks make based ports which require the autotool compile wrapper for cl.
In my point of view it is better to include/set those values directly in the toolchain.

Related: https://github.com/microsoft/vcpkg/issues/5721

@strega-nil is looking at this.
This is causing ignition-msgs1 and ignition-msgs5 to fail sporadically in CI.


Add integration test for ChanUpgradeAck step

Summary

Add tests for different scenarios for the ChanUpgradeAck step.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Closed by #3462


Secure email verification APIs

delete the /setEmailToVerified endpoint
unused + it allows anyone to verify their email without actually going through the process
refactor src/Pages/MembershipApplication/VerifyEmail.js, specifically the componentDidMount function to use the URL class, it helps with parsing the parameters in the url string, for example

const url = new URL("https://sce.sjsu.edu/verify?id=something&user=another")
url.searchParams.get("id") // returns "something"
url.searchParams.get("user") // returns "another"

completed with https://github.com/SCE-Development/Clark/pull/1331


Random status codes in the response to a request

Proposal
Is it possible for him to send a random status code to the same request? It is required to simulate the random behavior of the service so that it returns 200 or 400?
References
No response

First of all, I'm not sure if it's a good idea to do this as it will make your tests non-deterministic. But if that's the whole point, go ahead.
If you're writing your mocks in Java, this should be as easy as adding something like .withStatus(getRandomStatusCode()) where getRandomStatusCode() looks something like
public int getRandomStatusCode() {
    if(Math.random() < 0.5) {
        return 200;
    }
    else {
        return 400;
    }
}


Add arc_lock feature, with guards that can be accessed from inside of Arc

Resolves #273.
This adds guards for Mutex and RwLock that allow it to be used from inside of an Arc, without lifetime hurdles.

Great work! Can you update the CI script so that the arc_lock feature is tested?

It doesn't seem like you can use Arc in self without an MSRV bump, unfortunately.

It's fine I'll just bump the MSRV for the next release.


bibliography missing translator and editor

Description
some properties of a citation are missing from the long form of a bibliography entry while showing in the short form. I found the problem in the translator and editor properties, but it is likely to happen in multiple properties.
this is the bibtex file:
@book{test,
  title = {{title}},
  author = {author},
  translator = {translator},
  editor = {editor},
  year = {2024},
  series = {{series}},
  publisher = {publisher},
  address = {city}
}

the test file I used:
#set cite(style: "chicago-fullnotes")
@test
#bibliography("library_test.bib")

expected result:
Author. Title. Edited by Editor. Translated by Translator. Series. city: publisher, 2024.
actual result:

Reproduction URL
No response
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

When you write #set cite(style: "chicago-fullnotes") you're setting only the style for citations, not the bibliography (which here has defaulted to "IEEE"). If you want to set both, use #set bibliography(style: ) instead


BA-439 add height to position p tags

Something's wrong with this one. Can you pull develop? I'm getting no WP install on the multidev site
http://pr-362-borealis-ai.pantheonsite.io


Docs Changes

In the future, please perform tests like this on your own repositories instead of spamming those of others.
Thanks!


Use fbset to detect active framebuffer size, depth and dimension

Happened to find command fbset.
From Rpi 1B+ with GoodTFT 3.5" shows:
pi@raspberrypi:~ $ fbset

mode "640x480"
    geometry 640 480 640 480 32
    timings 0 0 0 0 0 0 0
    rgba 8/16,8/8,8/0,8/24
endmode

pi@raspberrypi:~ $ fbset -fb /dev/fb1

mode "480x320"
    geometry 480 320 480 320 16
    timings 0 0 0 0 0 0 0
    nonstd 1  # <-- wonder what this is..
    rgba 5/11,6/5,5/0,0/0
endmode

(Note geometry gives both resolution / bits depth)
This can be used to reduce # of init arguments drastically.
fbset defaults to fb0, regardless of FRAMEBUFFER env var, but could be decided by:

just iterate over fb0, fb1, ... - and use highest one mindlessly
If FRAMEBUFFER env var is set, then use it - otherwise default to fb0
make fb num argument positional, then use fbset to get it's details

Probably 3rd option sounds best for me.

Fixed via e2a77df6


⚠ TorrentLeech has degraded performance

In c17ae36, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 7431 ms

Resolved: TorrentLeech performance has improved in ae045b6 after 8 minutes.


docs: Fix "resource management" link

Looks like a broken link from how the previous version of the docs were structured

Resource management with dot as the link should automatically expand to an api link


Avro Trial for Bug with Unions AVRO-90

Avro Trial for Bug with Unions AVRO-90.

Hi!
Could you clarify what issue you're addressing with this PR? It looks like AVRO-90 was closed quite some time ago.


Branch IO Deep Linking works only after I clear the Safari history and cache data on iOS

I'm trying to add a sharing feature to my existing app using Unity Branch SDK. I retrieve custom data from initSession function of Branch SDK (Unity).
void Start()
{
    Branch.initSession(CallbackWithBranchUniversalObject);
}

void CallbackWithBranchUniversalObject(BranchUniversalObject buo,
                                    BranchLinkProperties linkProps,
                                    string error)
{
    string senderDeviceID = linkProps.controlParams["$deviceID"];
}

But initSession function seems to work a few times but it returns empty BranchUniversalObject JSON and BranchLinkProperties JSON after that. After a few days of struggling, I found that Safari history and cache data causes the issue. When I clear the Safari history and cache data, initSession function works again as I expected and I can retrieve custom data from Branch deep link.
How do Safari history and cache data affect the Branch deep link? What should I do to fix the problem of empty BranchUniversalObject JSON and BranchLinkProperties JSON?

Logging a ticket to see if we can repro the issue you're seeing.  Could you provide what version of the SDK you're using along with some information about the device you're seeing this on?  Also are you seeing the same issue on all test devices?

Thank you for your kind reply.
Branch Unity SDK
version: 0.6.5
Test Devices
Device Name: iPad Air2
iOS version: 13.4.1
I used the following code to generate a deep link.
public static void GenerateShortURL(BranchManagerCallbackWithShortUrl callback = null)
{
    BranchUniversalObject universalObject = new BranchUniversalObject();
    BranchLinkProperties linkProperties = new BranchLinkProperties();
    string deviceUniqueIdentifier = SystemInfo.deviceUniqueIdentifier;

    universalObject.canonicalIdentifier = "xLeagueShareLink";
    universalObject.canonicalUrl = "http://xleague.games/";
    universalObject.title = MetaDataManager.Title;
    universalObject.contentDescription = MetaDataManager.Description;
    universalObject.imageUrl = MetaDataManager.Image;
    universalObject.contentIndexMode = 0;
    universalObject.localIndexMode = 0;

    Branch.registerView(universalObject);

    linkProperties.feature = "sharing";
    linkProperties.controlParams.Add("$desktop_url", "http://xleague.games/xleague-solitaire-share/");
    linkProperties.controlParams.Add("$android_url", "http://xleague.games/android");
    linkProperties.controlParams.Add("$ios_url", "https://apps.apple.com/us/app/xleague-solitaire/id1480117114");
    linkProperties.controlParams.Add("$ipad_url", "https://apps.apple.com/us/app/xleague-solitaire/id1480117114");
    linkProperties.controlParams.Add("$og_title", MetaDataManager.Title);
    linkProperties.controlParams.Add("$og_description", MetaDataManager.Description);
    linkProperties.controlParams.Add("$og_image_url", MetaDataManager.Image);
    linkProperties.controlParams.Add("$deviceID", deviceUniqueIdentifier);

    Debug.Log("Starting get short URL...");

    Branch.getShortURL(universalObject, linkProperties, (param, error) =>
    {
        callback(param, error);
    });
}

Would you please help me?

Thanks for the information!  I'm hoping to devote the next sprint to working through github issues.

Please let me know if you need any other information to resolve this issue.

Hi, I'm facing the same issue... do we know if the fix is prioritized to be in the roadmap? Thanks!

Hi, does anyone has found any kinda of solution for this? I am still facing an issues


Allow custom path to kubeconfig and honor KUBECONFIG env var

Hey guys!
Neat tool, thank you! :)
I've started dabbling with it, and the first issue that I encountered is that it doesn't seem to be possible to use a non-default kubeconfig path. I.e, in my setup, I use multiple kubeconfig files via the KUBECONFIG env var:
❯ env | grep KUBE
KUBECONFIG=/Users/davidy/.kube/config:/Users/davidy/.kube/prod-config:/Users/davidy/.kube/dev-config:/Users/davidy/.kube/kubeconfig:/Users/davidy/.kube/multipass-config

However, kubescape doesn't seem to use the value of KUBECONFIG, and I couldn't find a commandline argument to explicitly specify it.
A --kubeconfig argument, and a reference to the KUBECONFIG env var would be much appreciated!
Cheers!
D

@funkypenguin Thank you for the helpful feedback :)
We will add support in a future version.

@mayocream Thank you for your contribution :)
@funkypenguin Thank you for your informative feedback, can you please confirm it is solved so we can close the issue?


Tower of Hanoi Problem in C++

Hey @Saikat2407 I would like to add a program to solve the Tower of Hanoi problem. So can you please assign me this?

Hey Can u assign me this issue , I have good knowledge in C++ programming langauge

@Saikat2407 could you please assign me, this task i would like to contribute.


Barely visible links in extension details webview using light themes

It looks surprisingly good in this screenshot, but in reality it's way worse. Contrast ratio:
1.6
background: #ebebeb, color: #a2c1e8

It doesn't make much sense to me. Background color changes ... color stays the same.
If background is close to blue (Solarized Dark) => links low contrast

If background is close to orange (Kimbie Dark) => code samples low contrast


🐛 Todos are off

So, to translate this into friendly language because you're talking to a person and not a machine: what behavior are you expecting, here?
Not everything you can do with obsidian is available in reveal.js.  It uses a different markdown parser/renderer (Obsidian is not involved in content rendering at all).
The elements are there for you to write CSS for your slides so that you can make them appear as you want them to, but the task behavior is not carried over.
For example, a list like this:
- [ ] one
- [x] two
- [/] three

Will be rendered in a slide like this:

<input disabled="" type="checkbox"> one

<input checked="" disabled="" type="checkbox"> two

[/] three


Add rogue point to mock dataset

Description:
Following the suggestion by @loforest, this PR adds an additional rogue point to the mock dataset to better illustrate the ampycloud diagnostic plot behavior with layers that have less than Theta_0 points in them.
Error(s) fixed:
Checklists:

[ ] New code includes dedicated tests.
[x] New code has been linted.
[x] New code follows the project's style.
[x] New code is compatible with the 3-Clause BSD license.
[x] CHANGELOG has been updated.
[ ] AUTHORS has been updated.
[ ] Copyright years in module docstrings have been updated.

@loforest, @regDaniel: here is the update mock example. I'll merge this into develop if you are ok with it ?


Molotov hull

bad

bad

bad

why ?
it's a helpful feature to see the exact extent of fire

bad

why ?
it's a helpful feature to see the exact extent of fire


Human-readable Parameters field

Hi,
I would like to know how to get human-readable parameters field? I used
transactionOp := content.(*rpc.TransactionOp)
but aint sure where to go from there to get human-readable parameters value. Please suggest thanks

Hi @echa ,
Thank you very much! it helps a lot. I think this example should be put in the README. It can help many to understand
Regards,
Ian


Remove ids from files and folders nodes

This was causing an issue and there is no reason for these nodes to have ids as they do not change state or are opened in the portal.
This will break until azuretools is updated to where id properties are optional.
Fixes #303

I'll just wait until we publish to merge it then


InFlightBatching seems not working

System Info

CPU: amd64
OS: Debian 12
GPU: nvidia rtx4000 ada
GPU driver: 535.161
TensorRT-LLM version: 0.8
tensorrtllm_backend version: 0.8

Who can help?
@kaiyux
Information

[x] The official example scripts
[ ] My own modified scripts

Tasks

[x] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
Followed exactly steps of: https://developer.nvidia.com/blog/turbocharging-meta-llama-3-performance-with-nvidia-tensorrt-llm-and-nvidia-triton-inference-server/. The only change is setting kv_cache_free_gpu_mem_fraction=0.95.
Expected behavior
Then I run 2 copies of https://github.com/triton-inference-server/tensorrtllm_backend/blob/main/inflight_batcher_llm/client/end_to_end_grpc_client.py in the same time. The first script will finish within 30 seconds. I expect the second one will finish around the same time (about 30 seconds)
actual behavior
The second will only finish after 60 seconds. Hence it seems like the batching is not working and every request will block other requests come after it.
additional notes
Very similar to #189, but the user reported that his issue was fixed after 0.6.1.

I am not sure how do you run the end_to_end_grpc_client.py. Could you try
python3 scripts/launch_triton_server.py --world_size 1 --model_repo=llama_ifb/ --log

echo '{"text_input": "What is machine learning?", "max_tokens": 20, "bad_words": "", "stop_words": "", "pad_id": 2, "end_id": 2}' > tmp.txt
printf '%s\n' {1..20} | xargs -I % -P 20 curl -X POST localhost:8000/v2/models/ensemble/generate -d @tmp.txt

and check the Active Request Count in the triton_log.txt?
Also, I see you are using the RTX 4000, which only has 8GB RAM. How do you follow the guid to run LLaMA 8B model?

@byshiue hi, it seems that I met the same issue as in #453 . Could you take a look at the issue? THX.

The issue is closed. You could also check the --max_num_tokens when build trt engine.


CAS Integration 2 - (UX/UI) Add or Edit CAS Supplier Info to Student Profile

User Story:
As a Ministry user, I want to see a student's supplier information on their profile, and be able to manually edit specific fields, so that I can view and manage their information and keep it current.
Acceptance Criteria:
Display CAS Info through UI Table

[ ] Create "CAS Supplier Info" tab in Student Profile. This is viewable to Ministry only (No institutions, no students)
[ ] Create "CAS Supplier Information" screen consistent with Fig 1. This table will function very similarly to the student's SIN table
[ ] Add text below the "CAS Supplier Information" header which reads "The top row in the table below is the most current CAS information for the student"
[ ] In response from CAS, look for first address/site info which has an active status, and Map those response variables to the UI table accoridng to the below mapping:

Column Header
Description/Mapping
E.g.

Date Created
Date When this row in the table was updated in SIMS
Sep 12, 2024

Supplier Number
supplierNumber
1234567

Supplier Protected
supplierprotected
Yes or No

Supplier Status
status
Active/Inactive

Site Code
suppliersitecode
001

Address Line 1
Addressline1 from CAS response
123 Fake St

Site Status
SupplierAddress > status
Active/Inactive

Site Protected
supplierAddress > siteprotected
Yes/No

[ ] Create a new line in this table everytime a 'Get Supplier' process returns a successful response

Allow CAS Info to be Manually Edited through UI

[ ] 'Edit Information' button should become available after a 'Get Supplier' transaction has been executed at least once.
[ ] Edit function is only permitted for 'Business Adminstrators' user role
[ ] If 'edit information' is submitted,

Both fields mandatory
Format & length validations on both fields, consistent with Individual Supplier Web Service Field Requirements (All Requests) JAN2023.xlsx
A new row in the table is created with just the manually inputted info (all other fields will be null)
A note is NOT created
NO further CAS calls/transactions are conducted

Business Context:

Handling 'update supplier site' and 'create new site' is in seperate ticket
Handling Get Supplier is in seperate ticket

Technical Context:
Mock-Ups
Fig 1

Fig 2 - Workflow

@CarlyCotton there is a note in the ticket about "A note is NOT created" but we believe that it would be important and it would also follow the SIN process. Is there a string reason to not have it?

@CarlyCotton the table is missing some record statuses highlighted below. Can we have those added to the table?

@CarlyCotton the below AC is not possible to achieve yet because we do not update the data if the Get Supplier did not return or did not return an active supplier.
"'Edit Information' button should become available after a 'Get Supplier' transaction has been executed at least once.".
To allow us to go ahead with this ticket, cn we move this AC to another ticket?

@andrewsignori-aot

Removed AC:

'Edit Information' button should become available after a 'Get Supplier' transaction has been executed at least once.

Updated Table to (hopefully match what you were telling me about

Going to get confirmation from business (finance) on the question of enforcing a note. I think there may be concerns about how limited the access to this CAS functionality/info is going to be and that it would be useful to a VERY small group of people.

@andrewsignori-aot

there is a note in the ticket about "A note is NOT created" but we believe that it would be important and it would also follow the SIN process. Is there a strong reason to not have it?

Confirmed with Finance, mandatory note NOT required. There are going to be a lot of limitations on who does this work and who can access this information. They can still add a note to the student profile if they need to.


Add automated tests

Create the automated tests

Added basic framework, but need more. See PR #11


Increment KSP version for stock TF pack.

This PR marks the latest version (0.2.0) of the TestFlightConfig-Stock-S42 config pack as compatible with KSP 1.9.

Cool!


Turbo Stream rendering plain text html instead of replacing content in targeted turbo frame

I am building an LLM backed app on Rails <IP_ADDRESS> where I want to continuously replace the main content based on the next prompt response. I start in home/index.html.erb which looks like this (some javascript which isn't shown, records audio and inserts it into the audio form field).

        Tell me something interesting you found out recently
        <button class="btn btn-primary speak-button">

        </button>

    <button id="record-button" class="btn btn-danger rounded-circle d-flex justify-content-center align-items-center" style="width: 100px; height: 100px;">

    </button>

    <%= form_with url: prompts_path, method: :post, data: { turbo_frame: :responses}, id: "audio-form", multipart: true do %>
        <%= file_field_tag :audio, style: 'display: none;', id: 'audio-file' %>
    <% end %>

    <%= turbo_frame_tag 'responses' do %>

    <% end %>

<%= javascript_include_tag 'audio_recorder' %>

So far so good. I click the record button and record my question, it stops recording and auto submits the form to /prompts.
My prompts controller looks like this:
class PromptsController < ApplicationController
    def create
        prompt_service = PromptService.new

        if params[:audio]
            audio = params[:audio]
            @transcription = prompt_service.transcribe_audio(audio)
            @subject = "all"
        else
            @subject = params[:title].downcase
            @transcription = params[:text]
        end

        gpt4_response = prompt_service.get_response_from_gpt4(@transcription, @subject)
        @responses = JSON.parse(gpt4_response)

        respond_to do |format|
            format.turbo_stream
        end
    end
end

It renders the prompts/create.turbo_stream.erb no problem and I see the responses I expect rendered correctly.
 <%= turbo_stream.update "responses" do %>
        <%= render "prompts/transcription", transcription: @transcription %>

        <%= render "prompts/responses", responses: @responses %>
<% end %>

_transcription.html.erb:

Transcription:

<%= transcription %>

_responses.html.erb:

     <% responses.each do |key, value| %>

             <%= form_with url: prompts_path(format: :turbo_stream), method: :post, class: "clickable-form" do |form| %>

                   <%= form.hidden_field :title, value: key %>
                   <%= form.hidden_field :text, value: value %>
                   <%= key %>

                 <%= value %>

             <% end %>
             <button class="btn btn-primary speak-button">

             </button>

     <% end %>

The html looks like this after the first render of the turbo stream
<turbo-frame id="responses">

Transcription:

Blue whales are the biggest animal to have ever lived.

              <form class="clickable-form" action="/prompts.turbo_stream" accept-charset="UTF-8" method="post"><input type="hidden" name="authenticity_token" value="QLvB_NtoFVpgQRGQmVtXrTHhpduPKUYTn4RTCQ0KozVb6dao7h-zL_JstuJ4KPutaTgUg" autocomplete="off">

                    <input value="Language and Literature" autocomplete="off" type="hidden" name="title" id="title">
                    <input value="Herman Melville's classic novel 'Moby-Dick' features a sperm whale as its central character, but blue whales are even larger than the fictional Moby-Dick and any sperm whale in reality." autocomplete="off" type="hidden" name="text" id="text">
                    Language and Literature

                  Herman Melville's classic novel 'Moby-Dick' features a sperm whale as its central character, but blue whales are even larger than the fictional Moby-Dick and any sperm whale in reality.

                </form>
                <button class="btn btn-primary speak-button">

              </button>

</turbo-frame>

The issue starts when I submit the form above from _responses.html.erb for a second roundtrip of the turbo stream. It renders plain text html of only the turbo frame content to the browser as in this image:

I have tried using various other turbo stream methods such as replace and advance. I have also tried using turbo frames with  a prompt/create.html.erb. This has similar behaviour in that the second time around it loads a whole new html page with just the turbo frame content. Headers and response formats etc all look correct to me (this is incorrect... see new comment).
If I copy the form structure from the working initial audio capturing form i.e adding data: { turbo_frame: :responses} and removing format: :turbo_stream I get ActionController::UnknownFormat in PromptsController#create
The audio capturing form continues to work across multiple submissions and re-renders of the turbo frame. I've just come across to Turbo from Stimulus Reflex and feel like I've read half of the internet trying to understand what's going on at this stage. I am not really sure where to go from here. Any advice or insights would be very much appreciated.

I've had a bit of time and headspace to look at this again and on second look realised the Request Header on the second form is not being set correctly. It is Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/ png,image/svg +xml,*/*:q=0.8 instead of Accept: text/vnd.turbo-stream.html, text/html, application/xhtml+xml.
Unless I am misunderstanding the implementation I suspect what is causing the issue is events on new forms being rendered from within a turbo stream are not being caught. Adding the format: :turbo_stream attribute doesn't seem to change this. This issue lead me here: https://github.com/hotwired/turbo/issues/1069 .
For now I have worked around this issue by populating the original form with extra fields and submitting that instead of having an alternative form as in the original code submitted.

don't add format: turbo_stream to your form url: prompts_path(format: :turbo_stream) should just be prompts_path. also you need to use requestSubmit() when submitting a form this.closest('form').requestSubmit().


ComputeSlopeV1

Calculates the slope between two points on an elliptic curve.
Reference: https://github.com/starkware-libs/cairo-lang/blob/efa9648f57568aad8f8a13fbf027d2de7c63c2c0/src/starkware/cairo/common/cairo_secp/ec.cairo#L53

I believe this is the right hint: https://github.com/starkware-libs/cairo-lang/blob/efa9648f57568aad8f8a13fbf027d2de7c63c2c0/src/starkware/cairo/common/cairo_secp/ec.cairo#L91

Sure! Assigned to the issue

Hi @cicr99 I opened a PR regarding this issue. Thanks

I close this issue as it has been completed with #377


ISPN-5931 Rename ContinuousQueryResultListener to ContinuousQueryList…

…ener
jira: https://issues.jboss.org/browse/ISPN-5931
This also deprecates ClusterRegistry.
Master only.

Seems trivial enough, pulling...

Integrated into master, thanks @anistor