Can't download a playlist. Console hangs then gives a TypeError: cannot unpack non-iterable NoneType object

Environment

System OS: MacOS Catalina Version 10.15.5

Python Version: Python 2.7.16 / Python 3.7.3

Install Source: Many different sources as I tried to reinstall it a bunch of times, but the last one was from git+https://github.com/nficano/pytube

Version OR Commit hash: Version: 3.1.4

Expected Behaviour
Expected to download a playlist

Actual Behaviour
Console hung for a while and gave me an error:
TypeError: cannot unpack non-iterable NoneType object

Steps to Reproduce

Clean up previous installation:

pip3-autoremove spotdl

Install spotdl from github:

pip install https://codeload.github.com/spotDL/spotify-downloader/zip/master
pip3-autoremove pytube
pip install git+https://github.com/nficano/pytube

Run it:

spotdl https://open.spotify.com/playlist/7JAdfJue28XcbNFaAM7xKf

??
profit

Traceback

Fetching Playlist...
Traceback (most recent call last):
  File "/usr/local/bin/spotdl", line 8, in <module>
    sys.exit(console_entry_point())
  File "/usr/local/lib/python3.9/site-packages/spotdl/__main__.py", line 111, in console_entry_point
    songObjList = get_playlist_tracks(request)
  File "/usr/local/lib/python3.9/site-packages/spotdl/search/utils.py", line 82, in get_playlist_tracks
    song = SongObj.from_url(
  File "/usr/local/lib/python3.9/site-packages/spotdl/search/songObj.py", line 66, in from_url
    youtubeLink = SongObj.searchProvider(
  File "/usr/local/lib/python3.9/site-packages/spotdl/search/provider.py", line 478, in search_and_get_best_match
    results = search_and_order_ytm_results(
  File "/usr/local/lib/python3.9/site-packages/spotdl/search/provider.py", line 366, in search_and_order_ytm_results
    results = __query_and_simplify(songSearchStr)
  File "/usr/local/lib/python3.9/site-packages/spotdl/search/provider.py", line 176, in __query_and_simplify
    linkBlock = contents['musicResponsiveListItemRenderer'] \
KeyError: 'playNavigationEndpoint'
Exception ignored in: <function tqdm.__del__ at 0x107e5cdc0>
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1126, in __del__
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1273, in close
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1458, in display
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1129, in __repr__
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/progressHandlers.py", line 23, in format_dict
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1416, in format_dict
TypeError: cannot unpack non-iterable NoneType object

Any other information:
Sorry for saying this here, but I really liked spotdl V2.
Sadly it doesn't work anymore, but I couldn't download a single song with spotdl V3.
If anyone knows how to get V2 working again, I'd be really happy :)

Somewhat related to #1025, but that person got another error, and then this one.
Tried all the fixes suggested on that conversation, one by one. None changed the error message I am getting.
Thought this deserved its own issue.

Just downloaded the album and the song from the example. Only playlists are no working

Exactly same error here on BigSur

It's happening for me also on Ubuntu 20.04

It's happening for me also on Ubuntu 20.04

@gordominossi Firstly: Any reason you have 2 versions of python installed? Stick to just using python 3 unless you have a reason to still have py2 installed. Also note that your python v3 is outdated.
EDIT: Your stacktrace uses python 3.9 - What information are you giving us at the top?
Your only stated install source was a pytube link. This is spotDL, not pytube. Please follow the issue template - You did provide this information later on in the issue, but I could not find it easily at the top.
If installed from master/github, we need a commit hash, to know when it was installed. pip show spotdl will not help us.
spotDL's original author, ritiek attempts to maintain a v2 fork. See it at ritiek/spotify-downloader, but it is currently dysfunctional, and has no support from the spotDL team.

@gordominossi Firstly: Any reason you have 2 versions of python installed? Stick to just using python 3 unless you have a reason to still have py2 installed. Also note that your python v3 is outdated.
EDIT: Your stacktrace uses python 3.9 - What information are you giving us at the top?
Your only stated install source was a pytube link. This is spotDL, not pytube. Please follow the issue template - You did provide this information later on in the issue, but I could not find it easily at the top.
If installed from master/github, we need a commit hash, to know when it was installed. pip show spotdl will not help us.
spotDL's original author, ritiek attempts to maintain a v2 fork. See it at ritiek/spotify-downloader, but it is currently dysfunctional, and has no support from the spotDL team.

Noted that this is actually a duplicate of #1038, and is fixed in commit 162d7ce
Comment below if still experiencing issues after installing from master again.

Noted that this is actually a duplicate of #1038, and is fixed in commit 162d7ce
Comment below if still experiencing issues after installing from master again.

Just built from master and still getting this error.

Just built from master and still getting this error.

Sorry if I was rude or something that caused any bad feelings on anyone. I'm just trying to help, reporting a bug I found.
I am using python 2 and 3 because I have programs that use python 2 and programs that use python 3. I said I use version 3.7.3 because that's what I get when I run python3 --version.
I have made a mistake when I wrote the bug report, pardon me. I tried installing spotdl from different sources and the last one was from pip3 install https://codeload.github.com/spotDL/spotify-downloader/zip/master.
Now I've tried from commit 162d7ce (latest commit of the master branch as of writing this comment) and got the same error message when trying to download a playlist.
Tried downloading a track and an album and both still work :)

Sorry if I was rude or something that caused any bad feelings on anyone. I'm just trying to help, reporting a bug I found.
I am using python 2 and 3 because I have programs that use python 2 and programs that use python 3. I said I use version 3.7.3 because that's what I get when I run python3 --version.
I have made a mistake when I wrote the bug report, pardon me. I tried installing spotdl from different sources and the last one was from pip3 install https://codeload.github.com/spotDL/spotify-downloader/zip/master.
Now I've tried from commit 162d7ce (latest commit of the master branch as of writing this comment) and got the same error message when trying to download a playlist.
Tried downloading a track and an album and both still work :)

Actually, different error, same message. Here's the sacktrace
Fetching Playlist...
Traceback (most recent call last):
  File "/usr/local/bin/spotdl", line 8, in <module>
    sys.exit(console_entry_point())
  File "/usr/local/lib/python3.9/site-packages/spotdl/__main__.py", line 113, in console_entry_point
    downloader.download_multiple_songs(songObjList)
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/downloader.py", line 88, in download_multiple_songs
    self.downloadTracker.load_song_list(songObjList)
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/progressHandlers.py", line 177, in load_song_list
    self.backup_to_disk()
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/progressHandlers.py", line 196, in backup_to_disk
    remove(self.saveFile)
TypeError: remove: path should be string, bytes or os.PathLike, not NoneType
Exception ignored in: <function tqdm.__del__ at 0x103789d30>
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1135, in __del__
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1282, in close
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1467, in display
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1138, in __repr__
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/progressHandlers.py", line 23, in format_dict
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1425, in format_dict
TypeError: cannot unpack non-iterable NoneType object

Actually, different error, same message. Here's the sacktrace
Fetching Playlist...
Traceback (most recent call last):
  File "/usr/local/bin/spotdl", line 8, in <module>
    sys.exit(console_entry_point())
  File "/usr/local/lib/python3.9/site-packages/spotdl/__main__.py", line 113, in console_entry_point
    downloader.download_multiple_songs(songObjList)
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/downloader.py", line 88, in download_multiple_songs
    self.downloadTracker.load_song_list(songObjList)
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/progressHandlers.py", line 177, in load_song_list
    self.backup_to_disk()
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/progressHandlers.py", line 196, in backup_to_disk
    remove(self.saveFile)
TypeError: remove: path should be string, bytes or os.PathLike, not NoneType
Exception ignored in: <function tqdm.__del__ at 0x103789d30>
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1135, in __del__
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1282, in close
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1467, in display
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1138, in __repr__
  File "/usr/local/lib/python3.9/site-packages/spotdl/download/progressHandlers.py", line 23, in format_dict
  File "/usr/local/lib/python3.9/site-packages/tqdm/std.py", line 1425, in format_dict
TypeError: cannot unpack non-iterable NoneType object

Appears to be a duplicate of #1044 then.

Appears to be a duplicate of #1044 then.

Thanks, will look into it

Thanks, will look into it

The commands below fixed my issue:
sudo pip3 install python3-pip-autoremove
pip3 install pip-autoremove
pip3-autoremove spotdl
pip3 install spotdl
spotdl https://open.spotify.com/track/7meYcjUYsTwiAZUp8nKIZl?si=fd11ffd15fe74fa9


French translation

French translation except for "buzzers" which I don't know how to translate. What is it for?
On my phone, pressing "Player A" activates the box, which deactivates after a few seconds, but there is no sound or vibration.

What is it for?

It's for e.g. a quiz game where it matters if person A or B first pressed the buzzer. I can add a sound and vibration there.

Thank you for the answer.
It can simply be translated as "Buzzer" then. I am not a quiz game player, I did not make the connection :)

Thanks!

Thanks to you for this app :)


Why used hashed partition?

Having read the docs for both counted and hashed partition, I fail to understand what the (if any) advantages of hashed partitioning would be?

For sufficiently large numbers of tests, hashed sharding produces roughly the same number of tests per bucket. However, smaller test runs may result in an uneven distribution.

This suggests it's equally good as counted partition at high counts, and worse at low counts. Is there an obvious upside I'm missing?

Thanks for your question!
The main advantage of hashed partitioning is that it is robust to filtering. No matter what filtering you specify, a test will always be assigned to the same hash bucket.
Also I guess hashes are the canonical way to do sharding. It would be a bit weird for nextest to not provide it as an option imho!

Thank you, good answer!


Change remap loading to be fatal during startup.

This also enables a missing "remap.config" to be treated as an empty file (that is, it "parses" successfully and yields no remap rules).
Closes #8785

Thanks for working on this @SolidWallOfCode.
I think we'll want to mark this for the 9.2.x project as well since the patch that introduced the issue went into 9.2.x.

[approve ci fedora]

Does this still allow a remap to be partially loaded when it's bad? IMHO that's still a very bad issue since it allows problems to crop up that are even worse than just not reloading the file, you could end up with all_services-1 not being able to serve

No. The partial loading still happens during startup, but ATS is then immediately shut down so it's irrelevant. Reloads continue to be atomic (all or nothing of the new configuration).

As such, my strong preference is that we back out this train of PRs from 9.2.x, and roll forward with 10.0.x

In that case, we should revert #7782 from 9.2.x and not cherry-pick this PR to 9.2.x. I'll create a PR to revert #7782 from 9.2.x.

I think @SolidWallOfCode has a strong preference to not revert.

I went back and checked. Prior to #7782 a missing "remap.config" was allowed. This is a reversion to previous behavior.
I have checked this in four cases to verify the operation is what is requested.

Valid, empty, missing "remap.config" at start up - successful start.
"remap.config" containing an error at start up - emergency shutdown (no restart).
Valid, empty, missing "remap.config" for reload - remap rules are updated.
"remap.config" containing an error for reload - remap rules are not updated.

This makes "remap.config" loading atomic - all of the file, or none of it.

Since ATS prior to #7782 has the features you want, can't we just revert #7782?  If not what is the difference in behavior now vs pre-#7782?

This change unifies start up and reload handling of "remap.config". It also changes startup failure to Emergency from Fatal to avoid a restart loop.  I've also tested this version and verified it behaves as requested.
However, due to some pushback, I plan to add a configuration variable to enable requiring a minimum number of rules in the "remap.config" to be valid. The default, at least for 9.2, will be 1. This means an empty or missing file won't replace a valid file. That can be changed by adjusting the configuration value. LinkedIn and Apple. I pondered a configuration for requiring a file but decided that was too indirect - a file with just comments is really an empty file. Valid rules are the correct metric for determining the "emptiness" of a file.

@bneradt is going to review this


parallel: Error: Command (--results) starts with '-'. Is this a wrong option?

Dear Jts,
I have successfully installed nanopolish and parallel (the latest versions), but when I run the parallel command, the following error was appeared.

When I ran the commands, parallel and nanopolish_makerange.py, separately —— it looks fine. I cannot figure out why they cannot be combined by '|'? Could you pls help me? Thank you!

Please post the full command you tried to run, I can't see the whole thing.
Jared
On Wed, May 16, 2018 at 4:39 AM, HuJane<EMAIL_ADDRESS>wrote:

Dear Jts,
I have successfully installed nanopolish and parallel (the latest
versions), but when I run the parallel command, the following error was
appeared.
[image: image]
https://user-images.githubusercontent.com/17492179/40105027-67720f5a-5924-11e8-9675-42d12880d065.png
When I ran the commands, parallel and nanopolish_makerange.py, separately
—— it looks fine. I cannot figure out why they cannot be combined by '|'?
Could you pls help me? Thank you!
[image: image]
https://user-images.githubusercontent.com/17492179/40106179-a427b5e6-5927-11e8-926c-0c93fb283587.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jts/nanopolish/issues/385, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXxn9DZR-ODkdzHPu4Fk8wWlYAd9oQgks5ty-WtgaJpZM4UA4Fr
.

Hi Jared,
My full command:
python /home/hujie/software/nanopolish/scripts/nanopolish_makerange.py canu/Normal.contigs.fasta | parallel --results nanopolish.results -P 6 nanopolish variants --consensus polished_Normal.{1}.fa -w {1} -r Normal.fasta -b Normal.sorted.bam -g canu/Normal.contigs.fasta -t 8 --min-candidate-frequency 0.1 --methylation-aware dcm,dam

$ python /home/hujie/software/nanopolish/scripts/nanopolish_makerange.py canu/Normal.contigs.fasta | parallel --results nanopolish.results -P 6 nanopolish variants --consensus polished_Normal.{1}.fa -w {1} -r Normal.fasta -b Normal.sorted.bam -g canu/Normal.contigs.fasta -t 8 --min-candidate-frequency 0.1 --methylation-aware dcm,dam
parallel: Error: Command (--results) starts with '-'. Is this a wrong option?
If commands ran separately:
python /home/hujie/software/nanopolish/scripts/nanopolish_makerange.py canu/Normal.contigs.fasta
tig00000012:0-891
tig00000025:0-388
tig00000047:0-662
tig00000048:0-727
tig00000052:0-1165
tig00000065:0-714
tig00000066:0-897
tig00000077:0-1213
tig00000171:0-615
tig00000253:0-1863
tig00000254:0-1176
tig00000255:0-545
tig00000256:0-1530
tig00000832:0-412
tig00000833:0-1148
tig00000834:0-686
tig00000835:0-1546
tig00000836:0-1027
tig00000837:0-1550
tig00000838:0-703
tig00000839:0-1206
tig00000840:0-836
tig00000841:0-446
tig00000842:0-366
tig00000843:0-475
tig00000844:0-479
tig00000845:0-1098
tig00000846:0-2790
$ parallel --results nanopolish.results -P 6 nanopolish variants --consensus polished_Normal.{1}.fa -w {1} -r Normal.fasta -b Normal.sorted.bam -g canu/Normal.contigs.fasta -t 8 --min-candidate-frequency 0.1 --methylation-aware dcm,dam
parallel: Input is read from the terminal. Only experts do this on purpose. Press CTRL-D to exit.

It looks like you are using a different version of parallel than we expect. You should be using GNU parallel. See this discussion here:
https://stackoverflow.com/questions/16448887/gnu-parallel-not-working-at-all

Your version of GNU Parallel is too old. --results was introduced in 20121022.

Yes, I used GNU parallel old version. After upgrade to 20150322, the problem is solved.
https://github.com/martinda/gnu-parallel
wget http://ftpmirror.gnu.org/parallel/parallel-20150322.tar.bz2
bzip2 -dc parallel-20150322.tar.bz2 | tar xvf -
cd parallel-20150322
./configure --prefix=${PWD} && make && make install


Organization Name
Hashnode
Your Name
Sandro Volpicella
Your current position
Fullstack Developer
(Optional) Company logo
https://drive.google.com/file/d/1UnN7rTXBrFYfG0HxnMxqKZP6Gg0-riYf/view?usp=sharing
(Optional) Use case
We use Lambda Powertools mainly for logging at the moment.
The logger package allows us to inject the context via middy of the lambda function and add additional correlation IDs such as user or post IDs.
Also using other Lambda Powertools languages?

[ ] Python
[ ] Java
[ ] .NET

Thank you @AlessandroVol23, appreciate you opening the issue!

The reference was added to the docs and to all the README files that are published on npm.
The docs are currently staged and will be published before the next release.
Thank you for supporting the project!


chore(main): release 1.24.0

:robot: I have created a release beep boop
1.24.0 (2023-09-27)
Features

post-install ffmpeg (#159) (c91f195)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aidanaden/canvas-sync/releases/tag/v1.24.0 :sunflower:


[Bug]: Docker registry credential not found using TESTCONTAINERS_HUB_IMAGE_NAME_PREFIX (regression in 4.0.0)

Testcontainers version
4.0.0
Using the latest Testcontainers version?
Yes
Host OS
Linux
Host arch
x86
.NET version
8
Docker version
[testcontainers.org 00:00:00.10] Connected to Docker:
Host: tcp://docker:2375/
Server Version: 27.3.1
Kernel Version: 5.4.0-187-generic
API Version: 1.47
Operating System: Alpine Linux v3.20 (containerized)
Total Memory: 15.62 GB
Docker info
[testcontainers.org 00:00:00.10] Connected to Docker:
Host: tcp://docker:2375/
Server Version: 27.3.1
Kernel Version: 5.4.0-187-generic
API Version: 1.47
Operating System: Alpine Linux v3.20 (containerized)
Total Memory: 15.62 GB
What happened?
Testcontainers stopped finding credentials for our custom docker registry in CI.
CI environment: gitlab CI, runners on k8s, we use dind as service to enable testcontainers in ci job.
We use custom docker registry on-prem (Harbor).
Credentials in CI stored in env var DOCKER_AUTH_CONFIG.
Relevant log output
[testcontainers.org 00:00:00.10] Connected to Docker:
Host: tcp://docker:2375/
Server Version: 27.3.1
Kernel Version: 5.4.0-187-generic
API Version: 1.47
Operating System: Alpine Linux v3.20 (containerized)
Total Memory: 15.62 GB
[testcontainers.org 00:00:00.12] Docker config "/root/.docker/config.json" not found
[testcontainers.org 00:00:00.12] Searching Docker registry credential in Auths
[testcontainers.org 00:00:00.12] Docker registry credential our.selfhosted.registry/dockerhub not found
...
Error Message:
Docker.DotNet.DockerApiException : Docker API responded with status code=InternalServerError, response={"message":"unauthorized: unauthorized to access repository: dockerhub/postgres, action: pull: unauthorized to access repository: dockerhub/postgres, action: pull"}
Stack Trace:
at Docker.DotNet.DockerClient.HandleIfErrorResponseAsync(HttpStatusCode statusCode, HttpResponseMessage response)
...
When we did rollback to 3.8
[testcontainers.org 00:00:00.09] Connected to Docker:
Host: tcp://docker:2375/
Server Version: 27.3.1
Kernel Version: 5.4.0-187-generic
API Version: 1.47
Operating System: Alpine Linux v3.20 (containerized)
Total Memory: 15.62 GB
[testcontainers.org 00:00:00.11] Docker config "/root/.docker/config.json" not found
[testcontainers.org 00:00:00.12] Searching Docker registry credential in Auths
[testcontainers.org 00:00:00.12] Docker registry credential our.selfhosted.registry found
[testcontainers.org 00:00:10.00] Docker image our.selfhosted.registry/dockerhub/postgres:15.7 created
[testcontainers.org 00:00:19.59] Docker container bf12b593f26c created
...
Additional information
Rollback to 3.8 made CI working again.

Oh, this is unfortunate. Could you please try removing /dockerhub from the auth node in the JSON value? It looks like we're resolving the hostname slightly differently, see our.selfhosted.registry/dockerhub versus our.selfhosted.registry. The DOCKER_AUTH_CONFIG should be something like this:
{"auths":{"our.selfhosted.registry":{"auth":"bXlfdXNlcm5hbWU6bXlfcGFzc3dvcmQ="}}}

Oh, this is unfortunate. Could you please try removing /dockerhub from the auths node in the JSON value? It looks like we're resolving the hostname slightly differently, see our.selfhosted.registry/dockerhub versus our.selfhosted.registry. The DOCKER_AUTH_CONFIG should be something like this:
{"auths":{"our.selfhosted.registry":{"auth":"bXlfdXNlcm5hbWU6bXlfcGFzc3dvcmQ="}}}

Our config is almost exactly the same as you posted:
{"auths": {"https://our.selfhosted.registry": {"auth": "****"}}}
dockerhub is the namespace or scope (not sure how it's called) where images stored. Consider: when you pull postgres:latest from dockerhub, you are pulling from library/postgres namespace/scope. Our selfhosted registry does the same, but with dockerhub if image doesn't have namespace, like confluentinc/cp-kafka:7.7.1.

I read it wrong. Try (I thought version 3.8.0 expected the "namespace," but version 4.0.0 does):
{"auths":{"our.selfhosted.registry/dockerhub":{"auth":"bXlfdXNlcm5hbWU6bXlfcGFzc3dvcmQ="}}}

I read it wrong. Try (I thought version 3.8.0 expected the "namespace", but version 4.0.0 does):
{"auths":{"https://our.selfhosted.registry/dockerhub":{"auth":"bXlfdXNlcm5hbWU6bXlfcGFzc3dvcmQ="}}}

Edit: How do you pass the image to the container builder? Do you use WithImage(IImage)?

I will try to change auth later, thats not as simple, the variable is shared across different projects in Gitlab.
We pass image this way:
_postgresContainer = new PostgreSqlBuilder()
            .WithImage("postgres:15.7")
            .WithEnvironment(..
            .WithDatabase(..
            .WithUsername(..
            .WithPassword(..
            .WithResourceMapping(
                new FileInfo(Path.Combine(
                    CommonDirectoryPath.GetProjectDirectory().DirectoryPath,
                    "custom-pg-docker-entrypoint.sh")),
                new FileInfo("./custom-pg-docker-entrypoint.sh"),
                UnixFileModes.UserRead | UnixFileModes.GroupRead | UnixFileModes.UserExecute | UnixFileModes.GroupExecute)
            .WithEntrypoint("./custom-pg-docker-entrypoint.sh")
            .WithCommand("")
            .WithEnvironment(..
            .Build();

Okay, I understand. You are using the custom configuration with the Docker Hub prefix. Now I can reproduce the issue. The following example fails as well:
public class GitHub
{
    static GitHub()
    {
        TestcontainersSettings.HubImageNamePrefix = "our.selfhosted.registry/dockerhub";
    }

    [Fact]
    public async Task Issue1286()
    {
        var dockerImage = new DockerImage("postgres:15.7");
        var container = new ContainerBuilder().WithImage(dockerImage).Build();
        Assert.Equal("our.selfhosted.registry", container.Image.Registry);
        Assert.Equal("our.selfhosted.registry", container.Image.GetHostname());
        await container.StartAsync();
    }
}

Assert.Equal() Failure: Strings differ
Expected: "our.selfhosted.registry"
Actual:   "our.selfhosted.registry/dockerhub"

Looks like something in this line breaks it.

Good to see you was able to find root cause. Thanks.

You are using the custom configuration with the Docker Hub prefix.

Yes, exactly. Sorry I forgot to mention. TESTCONTAINERS_HUB_IMAGE_NAME_PREFIX has this our.selfhosted.registry/dockerhub

Okay, we need to extend the DockerImageFixture test data and consider the substitution of the registry:
Add(new DockerImageFixtureSerializable(new DockerImage("postgres", null, null, null, "myregistry.azurecr.io")), "myregistry.azurecr.io/postgres", "myregistry.azurecr.io/postgres:latest");
Add(new DockerImageFixtureSerializable(new DockerImage("postgres", null, null, null, "myregistry.azurecr.io/library")), "myregistry.azurecr.io/library/postgres", "myregistry.azurecr.io/library/postgres:latest");

Basically, the overloaded constructor assigns the /library part to the Registry instead of the Repository property:

Actual
Expected

Registry
myregistry.azurecr.io/library
myregistry.azurecr.io

Repository
postgres
library/postgres

We need to be careful with the test assertion. The error message is reversed compared to what is shown in the table, which is incorrect. The table is correct.

Version 3.10 also seems to work fine FWIW.

The regression is due to the changes we made in the IImage interface (in version 4.0.0). I have a potential fix and will create a PR later today.

#1287 should fix the issue. It would be great if you could validate the fix and check if it’s working in your environment.

@digital88 #1287 should fix the issue. It would be great if you could validate the fix and check if it’s working in your environment.

Thanks, I think I can try validate the fix. Is there a nupkg artifact that can be installed? Or I should wait for patch release?

Thanks. I can publish a snapshot release by the end of this week. I would like to merge some changes, replacing the Docker client before.

The fix is included in: 4.1.0-beta.11746011829.

Can confirm 4.1.0-beta.11746011829 is working


Thanks for all your help! Can I buy you a coffee/beer/tea?

Hey, just wanted to show some support. Do you have Ko-Fi or something so I can buy you a drink? I really appreciate your responsiveness!

https://afdian.net/a/Chaoses-Ib/plan
I don't have an international card and can't register Ko-Fi. This platform says it supports PayPal and Stripe. I hope it works. It shows "/ Month" but it's not a subscription, it's a one-time payment, so don't worry.

after finding this: https://www.reddit.com/r/assholedesign/comments/1566g5a/afdian_a_chinese_website_doesnt_allow_users_to/ including the disclaimer at the top, that they can "terminate the cancellation with my consent" is garbage. It also sounds like it is a subscription, as that's what / Month means. Why use that terminology if it's not a subscription? Anyways, sorry, too sketchy.

if you have any other mechanism, let me know. E.g. direct through PayPal.

I do have a Paypal<EMAIL_ADDRESS>not sure if it can receive without linking any card.
After typing the above words, I tried again to connect my PayPal to Ko-fi, and this time it works instead of displaying some random error code. I thought the error was because I didn't link any card, but now it seems it's just a bug of Ko-fi/PayPal... Anyway, my page is here: https://ko-fi.com/chaosesib


make Web Scraper

Scraping from different websites using different python libraries

Can you assign this to me?

@sumanth-0 Hey I have completed this #21


Exclude RequestMapping parameters

How to exclude query params from the description when they are defined in @RequestMapping?
For example, the parameter "op" need to be omitted. I'm using springfox-2.5.0
    @ApiOperation(value = "list folder contents")
    @RequestMapping(
        params = "op=LIST",
        method = RequestMethod.GET,
        produces = MediaType.APPLICATION_JSON_UTF8_VALUE
    )
    @ResponseStatus(HttpStatus.OK)
    public Page<FileSystemItem> list() {
    }

Currently you cannot, but I see your point about being able to

@dilipkrish Thank you for your reply! Probably, one can work around this by explicitly introducing "op" as a controller method parameter and annotating it by @ApiIgnore.

@dmitrychief true. Plus you could set the defaultValue on the RequestParam to LIST and it would be equivalent.


Cannot query image from component field : Cannot query field "publicURL" on type "StrapiHomepageImg_descImage".

Hello,
I've got an error Cannot query field "publicURL" on type "StrapiHomepageImg_descImage". when I tried to get  the  publicURL of an image component( with an image field and name field)
img_desc{
  image{
    publicURL
  }
   name
}

But it's not happened all the times.
I thought it was a asynchronous problem. My server being not very fast, the response can be sometimes slower than usual..
So I checked the plugin , it seems that you forgot  await at 2 places, it should be await extractFields(apiURL, store, cache, createNode, touchNode, auth, f) to respect the asynchronous behavior.
https://github.com/strapi/gatsby-source-strapi/blob/2d5d641538c42d24dc2bf2c817139bdcc1f0e7e9/src/normalize.js#L18
https://github.com/strapi/gatsby-source-strapi/blob/2d5d641538c42d24dc2bf2c817139bdcc1f0e7e9/src/normalize.js#L69
I made the fix on a fork, and now everything works fine for me :)
Thank you very much for your fantastic Strapi !!

Hello @Jaikant ,
you're right, I've just commit the fix. Because the await was present on my computer so I didnt take care of it...
Please try again and tell me :)

@demiton I tried with your new commits, but all the data fetching fails. No nodes are fetched.

@Jaikant , sorry for that , your issue is probably different than mine..

@demiton I've created local npm package for gatsby-source-strapi from your fork and it seems to be working as expected. Nice work!
Could you please make pull request to strapi original repo from your fork?
It would be great to have this working out of the box.
Thank you!

@demiton Amazing work! Thank you, you have just saved a project! I have done the same as @radoslawhryciow and created a local npm package which solves my issue.
Would be great to see this merged!

@samOpenforce :)
@alexandrebodin What could I do to have these changes in the official package?

@radoslawhryciow We will need a PR as we are still a small team and have to focus on the feature developement until we reach the right balance before moving on to the "SDK" stuff

@alexandrebodin ok, thanks.
@demiton is this ok for you that I will make PR with missing awaits?

Hello,
Sorry for the delay, I was busy theses days.
@radoslawhryciow , happy to see that my post helps some else !
You can do make the PR of course :)
Thank you very much,
Cheers,

Hello guys :)
@radoslawhryciow,  @alexandrebodin , finally is there any PR planned to fix this issue ?

@demiton I will create PR today.

@alexandrebodin PR is ready https://github.com/strapi/gatsby-source-strapi/pull/135
@demiton and others :) Please add your review to this PR - maybe this will speed up merging changes to master !
Thanks!

My PR fixing this issue was merged today.
@alexandrebodin Thank you for merge. When my changes will be published to npm as a new package?

I will be reviewing everything in one week and release a bit after

Hello guys,
sorry Im not agree with your PR #135, extractFields is defined as an async function const extractFields = async... , so you always need to put await before (the await is like then for promise). Because the function async function is  a promise as @alexandrebodin  saids, if it is inside a promise.all, you don't need await. But If after tests, @radoslawhryciow you see that the the async after map is needed, Im pretty sure you must put await before the function !

Hi @demiton,
you're right. extractFields is declared with the async keyword, we should call it with the await keyword. Without await it is still working correctly because of Promise.all.
Anyway I would suggest to use await keyword here for code to be more self-documenting (extractFields is still async, could be declared in some other file etc.).
@alexandrebodin what do you think?

Hi,
Doing
await Promise.all(arr.map(async () => await sth()))

is actually buggy.
if you throw in the iterator function it will throw an unhandledRejection error because the map iterator is supposed to be sync
This code should look like
await Promise.all(arr.map(() => aFunction()))

const aFunction =  async () => {}

you should not make the iterator function an async one as there is nothing to catch it if it throws.

Hello,
@alexandrebodin , I'm ok with you. But @radoslawhryciow , did you said without map(async()=>... it is not working ?
Sorry, I haven't the time to test right now, Im going to holidays..

@demiton
// #1 not working
field.map(async f =>
    extractFields(apiURL, store, cache, createNode, touchNode, auth, f)
)

// #2 not working
field.map(f =>
    extractFields(apiURL, store, cache, createNode, touchNode, auth, f)
)

// #3 working - currently in master branch
field.map(async f =>
    extractFields(apiURL, store, cache, createNode, touchNode, auth, f)
)

// #4 working
field.map(async f =>
   await extractFields(apiURL, store, cache, createNode, touchNode, auth, f)
)

Subscribing to this as this is still throwing an error.

I am getting these errors Cannot query field "publicURL" on type ... when querying on an image field.
Sometime it happens and sometime it doesn't, sometimes it says I need to use url and sometimes publicURL.
Can't figure out why it happens or how to reproduce consistently.

Same problem, but only when I deploy it on Netlify.
strapiPost(slug: {eq: $slug}) {
      title
      content
      category {
          id
          slug
          title
      }
      coverPhoto {
        publicURL
      }
      book {
        title
        link
        description
        coverPhoto {
          publicURL
        }
      }
      seoDescription
      seoTitle
    }

Same problem, but only when I deploy it on Netlify.
strapiPost(slug: {eq: $slug}) {
      title
      content
      category {
          id
          slug
          title
      }
      coverPhoto {
        publicURL
      }
      book {
        title
        link
        description
        coverPhoto {
          publicURL
        }
      }
      seoDescription
      seoTitle
    }

Hi @juanoa  what version are you working with ?

Hi @juanoa  what version are you working with ?

These are my dependencies:
("gatsby-source-strapi": "^0.0.12")
"dependencies": {
    "bootstrap": "^4.5.3",
    "gatsby": "^2.26.1",
    "gatsby-image": "^2.5.0",
    "gatsby-plugin-google-gtag": "^2.5.0",
    "gatsby-plugin-manifest": "^2.6.1",
    "gatsby-plugin-offline": "^3.4.0",
    "gatsby-plugin-react-helmet": "^3.4.0",
    "gatsby-plugin-sharp": "^2.8.0",
    "gatsby-plugin-sitemap": "^2.9.0",
    "gatsby-source-filesystem": "^2.5.0",
    "gatsby-source-strapi": "^0.0.12",
    "gatsby-transformer-sharp": "^2.6.0",
    "prop-types": "^15.7.2",
    "react": "^16.12.0",
    "react-bootstrap": "^1.4.0",
    "react-cookie-consent": "^6.2.1",
    "react-dom": "^16.12.0",
    "react-helmet": "^6.1.0",
    "react-icons": "^4.1.0",
    "react-markdown": "^5.0.3"
  }

These are my dependencies:
("gatsby-source-strapi": "^0.0.12")
"dependencies": {
    "bootstrap": "^4.5.3",
    "gatsby": "^2.26.1",
    "gatsby-image": "^2.5.0",
    "gatsby-plugin-google-gtag": "^2.5.0",
    "gatsby-plugin-manifest": "^2.6.1",
    "gatsby-plugin-offline": "^3.4.0",
    "gatsby-plugin-react-helmet": "^3.4.0",
    "gatsby-plugin-sharp": "^2.8.0",
    "gatsby-plugin-sitemap": "^2.9.0",
    "gatsby-source-filesystem": "^2.5.0",
    "gatsby-source-strapi": "^0.0.12",
    "gatsby-transformer-sharp": "^2.6.0",
    "prop-types": "^15.7.2",
    "react": "^16.12.0",
    "react-bootstrap": "^1.4.0",
    "react-cookie-consent": "^6.2.1",
    "react-dom": "^16.12.0",
    "react-helmet": "^6.1.0",
    "react-icons": "^4.1.0",
    "react-markdown": "^5.0.3"
  }

We release the v1 alpha you should test it it should fix your issue :) I hope to move it to stable once we have enough feedbacks :)

We release the v1 alpha you should test it it should fix your issue :) I hope to move it to stable once we have enough feedbacks :)

We release the v1 alpha you should test it it should fix your issue :) I hope to move it to stable once we have enough feedbacks :)

Bug fixed at v1 alpha using the following query:
singleImage {
  localFile {
    publicURL
  }
}

Thanks!

We release the v1 alpha you should test it it should fix your issue :) I hope to move it to stable once we have enough feedbacks :)

Bug fixed at v1 alpha using the following query:
singleImage {
  localFile {
    publicURL
  }
}

Thanks!


Update tests

[x] tests and linter show no problems (npm t)
[x] tests are added/updated for bug fixes and new features
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/metautil/commit/d0fee5698465612b6d8243c7be1d34c5bff3afc2 and https://github.com/metarhia/metautil/commit/cbbd8da716122bf57a13b4baa9252a69efa3848f


Slippage changes

Slippage support to smart contract
Swap and LP Token contract moved to int256
Added tests for various scenarios

why int instead of uint.
and why isn't that a separate PR. that stuff can lead to a lot of issues I think
you also down-leveled a int64 to int. AND you don't seem to be checking for overflows
int adjustedValue = (invariantValue * precision)/ (tokenBQ - amountTokenB);
^^ can this be abused by putting in REALLY large numbers and causing overflows.
do you have tests for overflows?
and also.. PLEASE use separate PRs for this going forward (as I can't even see the slippage stuff yet, as all this uint/int/int64 stuff if making it muddy)

Context why we are moving to int from int64 -

To avoid overflow as precision support is added to the smart contract.
Smart Contract will accept quantities in the multiple of precision. The precision at present is 10,000,000. But we are flexible to increase precision if the situation demands.
As smart contract accept multiple of precision as input, all the calculations e.g. multiplication or division requires a bigger data type hence int was chosen.
uint Vs int -  int was selected as smart contract  can still return negative numbers to clients if the functionality demands in future. Where uint is required because API dependencies like HTS, we can easily int to uint without truncation.


Table: overscanRowCount in the reverse of scrolling direction

Background
I'm working through an issue where one of my team's UI components that uses the react-virtualized table under the hood is making rows disappear from the top of the table as you scroll down the page. I'm presuming the real crux of the issue is with our team's implementation/usage of WindowScroller and AutoSizer, but even by playing with passing through the overscanRowCount to match the static number of rows in my table only appears to work in the scrolling direction - which is documented, so I'm not upset about it.
(Aside: Obviously, since I mentioned my row count is static, the real solution is "well, wait, why are you using react-virtualized then?" but I've been told that for maintainability, I should conform to our UI library and enhance that instead of going rogue and just putting together a

 for the application I'm working on. I really could go on about this, but this feature request shouldn't be coupled to my team's internal politics 😄. )
I'm hoping that this feature request is relatively quick to implement as I imagine the logic would mirror that of the existing overscanRowCount and if I find the time to dive into your codebase, I could try whipping up a PR myself, but that'd be "somewhat near, but not immediate future".
Formal Feature Request information

Use case: Application with a table with a known maximum row quantity will occasionally not render uppermost rows of table when scrolling down, and vice versa. A stopgap for "reverse overscanRowCount" would solve the problem at a (hopefully) smaller time cost than AutoSizer implementation debugging.

Proposed interface:

Similar Functionality Example: Can't think of any off the top of my head, sorry.

Also, for my specific use case, this would work:

which of course generally nukes the performance and should not be recommended at all, but would also be (presumably?) easy to implement as you'd just effectively set overscanStartIndex: 0, overscanStopIndex: rows.length && rows.length-1

Just figured I'd update here, in case anyone else is running into the same issue for the same reasons as me.
After looking through some of the codebase for the preliminary steps towards writing a PR, I found and was able to use the undocumented/almost-definitely-not-kosher prop overscanIndicesGetter, which - when I tell our "wrapper component" I need to __dangerouslyOverscanAllRows, I pass as:
additionalTableProps.overscanIndicesGetter = function({
                cellCount,
                overscanCellsCount,
                scrollDirection,
                startIndex,
                stopIndex
            }) {
                return ({
                    overscanStartIndex: 0,
                    overscanStopIndex: cellCount-1
                });
            };

Whoops, didn't mean to close just because I'd worked around my problem; clicked the wrong button. Reopening.

@JonathanMerklin I apologize for not getting to this earlier. Thanks for posting the work around.
However, I think we can also be a little big greedier when rendering rows in the opposite direction of scroll.

I get the same issue right now...but only when the number of rows is small i.e. smaller than a "page"


redirect_mismatch v0.20.0

hello,
i'm new to restish. i installed it and configured it the same as a colleague
$ cat ~/.config/restish/apis.json
{
  "$schema": "https://rest.sh/schemas/apis.json",
  "cp": {
    "base": "https://some_aws_api_gateway_url.eu-west-1.amazonaws.com/dev",
    "profiles": {
      "default": {
        "auth": {
          "name": "oauth-authorization-code",
          "params": {
            "authorize_url": "https://some_aws_cognito_endpoint.eu-west-1.amazoncognito.com/oauth2/authorize",
            "client_id": "some_client_id",
            "scopes": "openid email profile cp/api",
            "token_url": "https://some_aws_cognito_endpoint..eu-west-1.amazoncognito.com/oauth2/token"
          }
        }
      }
    },
    "tls": {}
  }
}

his set up worked, mine didn't. it looks like it's an issue with version 0.20.0 as he was running version 0.19.0
$ ./restishv19 --version && rm ~/.cache/restish/cache.json
restishv19 version 0.19.0
$ ./restishv19 cp

the above works fine but
$ ./restishv20 --version && rm ~/.cache/restish/cache.json
restishv20 version 0.20.0
$ ./restishv20 cp

produces the following error in the browser

which OS do you two use? to looks like it may be an duplicate of the issue I filed recently #248.
btw: amazon cognito showing such an error typically indicates that your requests mismatch with what cognito’s oidc idp server expects. in my the referenced issue above, i found missing http query parameter (when restish fired the authorize requested) also yielded a similar error page as you’re facing.

@DominicBortmes we use Fedora. the issue seems to be related to the trailing / on the redirect url. the default was changed in v0.20.0. i've created https://github.com/danielgtaylor/restish/pull/250 but in the meantime the workaround for me is just to add
            "redirect_url": "http://localhost:8484/"

to my config


Works great but when update model I need to refresh.

Hi,
the module works great but if I update model with a service the text doesn't fit.
I need a polish way to resize after update model.
I try with resize event but for some reason I can't use this.

Hi guys,
any update ? :-)
thanks a lot !

Hi! I'm really busy in these days, I'll promise I will fix it in September :)
If you or anyone have a solution, can submit a merge request.

Hi,
Please check this PR
https://github.com/lokenxo/ng2-fittext/pull/10
thanks


CLI to copy Admin Console images to private registry

What type of PR is this?

type::feature
What this PR does / why we need it:
This is required in order to do auto-upgrades when Admin Console images are used with a private registry.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
This CLI is hidden and will not be documented yet.  Usage looks like this:
$ ./bin/kots admin-console push-images docker.io ttl.sh/divolgin
Copying minio/minio:RELEASE.2022-01-08T03-11-54Z to ttl.sh/divolgin/minio:RELEASE.2022-01-08T03-11-54Z
Copying blob 0e3d9ac748ab skipped: already exists
...
Copying config c0092c1ed2 done
Writing manifest to image destination
Storing signatures
Copying kotsadm/kotsadm-migrations:v1.59.2 to ttl.sh/divolgin/kotsadm-migrations:v1.59.2
...
Copying config 0ed0ea7b7c done
Writing manifest to image destination
Storing signatures
...

Does this PR introduce a user-facing change?

* Added experimental CLI for copying Admin Console images to a private registry

Does this PR require documentation?

NONE

Another ngrok failure:

Page 'https://cd85606660c8.ngrok.io/' is not available


412

Blank space is missing in the first sentence.
It was during my first  flight in the balloon that I was instantly bitten by the flying
Sh! Let's not talk in this room. They might have put a $@$ to listen in on our conversations.
To stay home, little Mike pretended he had a nasty stomach $@$, but his mother didn't fall for that trick.

fixed in export from 08.03


Bundle improvement

compser cannot found latest tag, trying up version
fixed typo in composer.json
catch logger error if service down
do not try to reinitialize FluentLogger if it failed once
disable logging if host is not provided
Removed folder 'bundle' from namespace
Removed IDE specific files
Renamed class names to correct Symfony pattern

Thanks, I will review asap

More changes

wow, thanks. Away from office, will try to check later.

Hi,
The last two commits (e09cf08 and a1aa967) seem to actually break stuff.
In my point of view, the tag does not always need to contain the machine hostname.
I believe that respecting what the user configured as tag might be more generally useful.

Guys, I'm not using this bundle anymore, anyone want to take ownership?
I have too many side projects to maintain this.


More semiring algebras

From book: "Machine Learning: a probabilistic perspective"
How to proceed

The number types should be implemented by making a pull request to:
https://github.com/TensorBFS/TropicalNumbers.jl
Then we register a new version of TropicalNumbers.
Bind the tropical matrix multiplications over CuArrays to LinearAlgebra.mul!

Closed by #9


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1024

Questions ?
Please refer to the Backport tool documentation


Some processes don't register in list-apps or list-windows

Some processes/windows don't get picked up by Aerospace.
This is not that surprising for some processes - I encountered this with mpv and given that it doesn't even come close to using a macOS-native GUI, it's not shocking. But, IINA is supposed to be a macOS-native GUI frontend for mpv and also doesn't show up.
With both apps showing windows in the background, neither show up in aerospace's query commands:
$ iina && mpv --player-operation-mode=pseudo-gui & aerospace list-apps && echo && aerospace list-windows --all
[1] 13242
14266 | com.apple.finder                  | Finder
14437 | com.apple.Safari                  | Safari
14720 | net.kovidgoyal.kitty              | kitty
75355 | com.apple.iCal                    | Calendar
75783 | com.apple.Music                   | Music
81677 | com.apple.mail                    | Mail
85755 | com.microsoft.rdc.macos           | Microsoft Remote Desktop
59055 | com.apple.Notes                   | Notes
25845 | com.apple.MobileSMS               | Messages
34649 | com.google.Chrome                 | Google Chrome
38343 | com.apple.Preview                 | Preview

5297  | Calendar | Calendar
14962 | Finder   | Desktop
14911 | Mail     | Yesterday – 2 messages
1371  | Safari   | New Issue · nikitabobko/AeroSpace
1552  | kitty    | tmux - zsh
$

Checklist

[ X ] I've searched for duplicates. My search queries were (please don't forget to search in closed issues too):

list-apps
list-windows
...

[ X ] I've read the guide https://nikitabobko.github.io/AeroSpace/guide
[ X ] I've searched in the documentation (https://nikitabobko.github.io/AeroSpace/guide & https://nikitabobko.github.io/AeroSpace/commands). My search queries were:

process
window
...

aerospace CLI client version: 0.14.2-Beta 0cb8dbdfc5ee73b8cbc200f175f467ebead55201
AeroSpace.app server version: 0.14.2-Beta 0cb8dbdfc5ee73b8cbc200f175f467ebead55201

I just noticed the same. I was wondering why few of my windows didn't get tiled properly. This has happened a few times before, but now I by chance checked list-windows. Neither was there.
Restarting seems to have the issue once more. I tend to always start these two from the Dock, so there's probably something else at play as well than just the method of starting.
The two in question, WezTerm and Neovide, do seem like well-behaved macOS apps as well. :)

I hope you don't mind me commenting on this issue since I’m experiencing the same issue with Neovide when launching it from the terminal. I can’t even get the app to respond to any Aerospace shortcuts, almost like it doesn’t recognize the app at all.
the aerospace list-windows --all command sees neovide.
945 | Arc     | Some processes don't register.
147 | WezTerm |  ~/.config/nvim
113 | WezTerm |  ~/repos/projectsPort/newFormatCL
956 | neovide | Neovide
966 | neovide | Neovide

but aerospace list-apps doesn't as it sees neovide as NULL
444  | com.apple.finder           | Finder
831  | com.github.wez.wezterm     | WezTerm
9365 | company.thebrowser.Browser | Arc
9527 | NULL-APP-BUNDLE-ID         | neovide
9711 | NULL-APP-BUNDLE-ID         | neovide

@ashleyharvey  Thanks for the quick response and helpful info! It was a bit of a hectic process since this was my first time doing something like this.
In the end, it worked with just a few steps, thank you so much! It finally solved a problem I’ve been dealing with for a long time.

This seems to be occurring with me on the most recent update. Previously Ghostty, Spark and Outlook would be adopted by aerospace, but not they aren't. None of those three while open register in list-windows --all or list-apps.

This is running in Ghostty and Ghostty isn't listed. Though Spark and Outlook also should be listed. Previously all three of these worked.


Kulfy: Watch texaslocksmithandsecurity.com

Kulfy requests the watch of the watch_keyword texaslocksmithandsecurity\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

texaslocksmithandsecurity\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


where is the exe file?

i cant find the exe file, is that supposed to be generated by myself based on the souce code?

You can either build the executables from source yourself or get them from https://github.com/WombatFromHell/OriginSteamOverlayLauncher/releases

okay, how to deal with the easy anti cheat program, it is not working when i try to open an epic game, after the easy anti cheat window pumps out, and the game is not running

Which EGL game are you trying to launch? Check the project Wiki (tab up above) for extra instructions to aid in configuring OSOL for EGL.

yeah i am trying the url method for that, by the way, the name of the game is called Dauntless.

I just tried Dauntless, it's working perfectly. Make sure you've configured the OriginSteamOverlayLauncher.ini correctly. Additionally, you need to make sure that the OriginSteamOverlayLauncher.exe executable is NOT in the Dauntless Game folder (as stated on the wiki), or that will prevent the game from launching.
LauncherPath=C:\Program Files (x86)\Epic Games\Launcher\Portal\Binaries\Win64\EpicGamesLauncher.exe
GamePath=C:\Program Files\Dauntless\Archon\Binaries\Win64\Dauntless-Win64-Shipping.exe
LauncherArgs=
LauncherURI=com.epicgames.launcher://apps/Jackal?action=launch&silent=true

I just tried Dauntless, it's working perfectly. Make sure you've configured the OriginSteamOverlayLauncher.ini correctly. Additionally, you need to make sure that the OriginSteamOverlayLauncher.exe executable is NOT in the Dauntless Game folder (as stated on the wiki), or that will prevent the game from launching.
LauncherPath=C:\Program Files (x86)\Epic Games\Launcher\Portal\Binaries\Win64\EpicGamesLauncher.exe
GamePath=C:\Program Files\Dauntless\Archon\Binaries\Win64\Dauntless-Win64-Shipping.exe
LauncherArgs=
LauncherURI=com.epicgames.launcher://apps/Jackal?action=launch&silent=true


Add 'on Exception' to catch(e)

Pokémon exception handling caused the try...catch at line 401 to encounter an unexpected error when /QOpenSys/pkgs/bin/ directory path does not exist. The command ls -p /QOpenSys/pkgs/bin/ returns code 2. As a result, the next path for remoteApps /usr/bin/ is never checked for installed applications. This caused a false-negative for autoConvertIFSccsid in situations where it should have been usable.
Changes
Replaces a catch clause with a catch on Exception clause.
Checklist

[ ] have tested my change
[ ] updated relevant documentation
[ ] Remove any/all console.logs I added
[ ] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[ ] for feature PRs: PR only includes one feature enhancement.

"on Exception" is not valid syntax in a try...catch in JavaScript. Seeking alternatives.


Fix CMS password login issues

This PR fixes #21259
Before:

Behavior with fix:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 41.978%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
28754

Relevant Lines:
63636

💛  - Coveralls

@cheenamalhotra are you still planning to do something with this PR?  Or should we close until you're ready to revisit?

Let me take a look at it again today now that I know a bit more in this space.. I'll close it later today if it needs significant time.


Process packets only on the POSTROUTING hook (before SNAT)

We only need to process packets that are destined to Pods, never destined to the local host.
Based on the netfilter diagram https://wiki.nftables.org/wiki-nftables/index.php/Netfilter_hooks this can be on FORWARD and POSTROUTING.
However, FORWARD does not work with IPVS, as it seems the IPVS kernel logic follows a different path http://www.austintek.com/LVS/LVS-HOWTO/HOWTO/LVS-HOWTO.filter_rules.html
Using OUTPUT forces us to discriminate the traffic that is destined to localhost to avoid being impacted by network policies, usually adding a rule to accept packets to lo.
Using POSTROUTING before SNAT happens (to avoid to loose the original source IP) seems the right place
Fixes: https://github.com/kubernetes-sigs/kube-network-policies/issues/46

hmm, ipv6 flakes now 🤔

2 / 2 green with this PR https://github.com/kubernetes-sigs/kube-network-policies/actions
Accepting always ICMPv6 ND seems to make it work reliable

Beside the nit this lgtm. For documentation, this is how it looks now:
# nft list table inet kube-network-policies
table inet kube-network-policies {
        comment "rules for kubernetes NetworkPolicy"
        set podips-v4 {
                type ipv4_addr
                comment "Local V4 Pod IPs with Network Policies"
        }

        set podips-v6 {
                type ipv6_addr
                comment "Local V6 Pod IPs with Network Policies"
        }

        chain postrouting {
                type filter hook postrouting priority srcnat - 5; policy accept;
                icmpv6 type { nd-neighbor-solicit, nd-neighbor-advert } accept
                ct state established,related accept
                ip saddr @podips-v4 queue to 100 comment "process IPv4 traffic with network policy enforcement"
                ip daddr @podips-v4 queue to 100 comment "process IPv4 traffic with network policy enforcement"
                ip6 saddr @podips-v6 queue to 100 comment "process IPv6 traffic with network policy enforcement"
                ip6 daddr @podips-v6 queue to 100 comment "process IPv6 traffic with network policy enforcement"
        }
}

I tested e2e in dual-stack with IPv6 as "base family" on K8s v1.30.3.

An unrelated comment: when building a bin/ dir is created. A .gitignore with "/bin" would be appropriate IMHO.

Thanks Lars, do you mind following up on those nits?
I'd like to get a release with ipvs working and the nftables initialization fix

This appears to break my setup with kube-proxy in nftables mode and flannel with EnableNFTables (which enables masquerading in POSTROUTING).
I haven't had time to dig in to the issue yet but figured I'd leave a comment in case this wasn't tested with kube-proxy in nftables mode.

I can confirm, e2e test with FOCUS=Netpol fails with proxy-mode=nftables on main. I will check more tomorrow. I hope it is enough to change the chain priority (based on the fact that proxy-mode=iptables works with iptables in "nft-mode").

Once we have support in kind we can have CI in Kubernetes master https://github.com/kubernetes-sigs/kind/pull/3612

@vaskozl Can you please write an issue? Not too elaborate, basically what's in https://github.com/kubernetes-sigs/kube-network-policies/pull/54#issuecomment-2243647926.

This is how it looks with proxy-mode=nftables (narrowed to postrouting chains):
table ip kube-proxy {
      chain nat-postrouting {
              type nat hook postrouting priority srcnat; policy accept;
              jump masquerading
      }
      chain masquerading {
              meta mark & 0x00004000 == 0x00000000 return
              meta mark set meta mark ^ 0x00004000
              masquerade fully-random
      }
}
# (same for ip6)
table inet kube-network-policies {
      chain postrouting {
              type filter hook postrouting priority srcnat - 5; policy accept;
              icmpv6 type { nd-neighbor-solicit, nd-neighbor-advert } accept
              ct state established,related accept
              ip saddr @podips-v4 queue to 100 comment "process IPv4 traffic with network policy enforcement"
              ip daddr @podips-v4 queue to 100 comment "process IPv4 traffic with network policy enforcement"
              ip6 saddr @podips-v6 queue to 100 comment "process IPv6 traffic with network policy enforcement"
              ip6 daddr @podips-v6 queue to 100 comment "process IPv6 traffic with network policy enforcement"
      }
}

The netpol chain already has a higher prio srcnat - 5, which was what I intended to check, and this looks ok to me.
@vaskozl What problem do you see? The netpol e2e is only cluster internal (I think). Do you have problems with external traffic?

Sorry I wasn't sure there was an issue and didn't have details hence I didn't want to create a potentially false issue if nftables tested successfully:
In addition to the two tables you listed, I have the following postrouting priority srcnat; from flannel:
table ip flannel-ipv4 {
	comment "rules for flannel-ipv4"
	chain postrtg {
		comment "chain to manage traffic masquerading by flannel"
		type nat hook postrouting priority srcnat; policy accept;
		meta mark 0x00004000 return
		ip saddr <IP_ADDRESS>/24 ip daddr <IP_ADDRESS>/16 return
		ip saddr <IP_ADDRESS>/16 ip daddr <IP_ADDRESS>/24 return
		ip saddr != <IP_ADDRESS>/24 ip daddr <IP_ADDRESS>/16 return
		ip saddr <IP_ADDRESS>/16 ip daddr != <IP_ADDRESS>/4 masquerade fully-random
		ip saddr != <IP_ADDRESS>/16 ip daddr <IP_ADDRESS>/16 masquerade fully-random
	}
}

I tried upgrading twice and within seconds I see failed liveness probes across all pods and failed image pulls. External connections from within the cluster appear to timeout too. Downgrading required full node restarts.
It is a very weird issue indeed, given that for some reason I required a restart (I did check that the forward rules were reapplied on revert). I can look to debug more in the weekend and will create a propper issue with details if I find anything wrong.

rry I wasn't sure there was an issue and didn't have details hence I didn't want to create a potentially false issue if nftables tested successfully:

no worry about that, feel free to open issues like that to discuss, I'm not worried about issues that turn out to be environment problem for people like you that spend the time to discuss and to troubleshoot, this feedback is very welcome.
I'm going to setup a job to test nftables so we can rely on automated testing to validate nftables functionality

failed image pulls

This is interresting. It might be that the netpol postrouting chain interfere in some way with external traffic. I make sense to me since postrouting should affect pod-IP -> non-pod-IP traffic only.
I tested with the versions below, and Netpol e2e passes.
  11:35:48 Server Version: v1.30.3
  11:35:48 CNI-plugin; flannel:v0.24.2
  11:35:48 Proxy-mode: "nftables"
  11:35:48 crio version 1.29.2
  11:35:48 BaseFamily=IPv6

:thinking: but then external traffic problems should be a v0.5.0 issue, rather than an nftables issue. I'll make some tests with outgoing traffic from pods. I have no image load problems though.

@vaskozl What kind of netpol rules do you use? Most logical to me is that egress blocking rules would be causing issues for external traffic, but I may be wrong. In my own tests I use ingress blocking rules, so I want to extend with relevant rules.

I added TCP liveness/readiness probes, and if I apply my "deny ingress" rule, the probes fail. This make sense to me, a deny ingress should block the probes. @aojea Do you know if there is some "don't do that" for this situation documented somewhere. Please, don't search for it, I can do that myself, but if you happen to know...

I don't have any egress rules but apply traditional (not banp/anp) ingress netpols to most everything in the cluster. Afaik Host to local workload traffic should not be blocked by network policies and isn't by all CNIs that support netpols.
I use 1.30 with flannel in 0.25.3 and ipv4 single stack. I'll make an issue after I debug asap.

I backed down to v0.4.0, and TCP probes works with a "deny-all-ingress" rule applied.
@aojea I think this is the root of the problem

Afaik Host to local workload traffic should not be blocked by network policies and isn't by all CNIs that support netpols.

You are right.

I can't explain the failing image loads though.

That may have been a knock on of the whole cluster liveness probes causing restarts and some rolling restarts I issued that slowed down my kublet and as such and indirect observation on my relatively weak setup. Otherwise maybe hostNetwork pods that I also have rules on? I'll try to have more accurate information about the failure soon.

I added TCP liveness/readiness probes, and if I apply my "deny ingress" rule, the probes fail. This make sense to me, a deny ingress should block the probes. @aojea Do you know if there is some "don't do that" for this situation documented somewhere. Please, don't search for it, I can do that myself, but if you happen to know...

@danwinship is the source of truth for network policies 😄  and has also revisited the problem of probes recently, it also mentions this in the KEP https://github.com/kubernetes/enhancements/pull/4558
Changing the definition of NetworkPolicy v1 to remove the "kubelet
  can access all local pods" hole, even in the case where the cluster
  is configured in a way to forbid the use of probes that require the
  hole. Even if _kubelet_ no longer needs to access pods, some users
  likely depend on that rule to allow access to some pods from other
  host-level processes, and there is no other way in NetworkPolicy v1
  to write such a rule if we remove the built-in one

I think that traffic originated from the node should not be subject for network policies.
I checked with nftables trace and all trafic coming from the node has the iif lo independently that the destination IP belongs to another interface
trace id 2da4b9ff ip filter INPUT packet: iif "lo" @ll,0,112 0x800 ip saddr <IP_ADDRESS> ip daddr <IP_ADDRESS> ip dscp cs0 ip ecn not-ect ip ttl 64 ip id 30324 ip length 60 tcp sport 41872 tcp dport 10250 tcp flags == syn tcp window 33280

For the record; this is not a proxy-mode=nftables problem

yeah, opened a new issue so we have this well documented https://github.com/kubernetes-sigs/kube-network-policies/issues/65 , please continue the conversation there, also submitted a PR and tagged you both for feedback


SUBMARINE-327. Fix unchecked call 'result' when used JsonResponse.Builder build response

What is this PR for?
Fix the check warning for JsonResponse, the two warning problems list as below:

Unchecked call to 'result(T)' as a member of raw type
Raw use of parameterized class

What type of PR is it?
Bug Fix
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-327
How should this be tested?
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@jiwq Thank you fix this issue.
Will merge if no more comments.


Update LAB_06_ImplementDirectorySynchronization.md

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by ParamRamki on behalf of MOC Labs Review


AWS Security Hub parser add unique_id_from_tool and deduplication

Add name of the AWS resource in the title for ease of use and to help in the remediation process
Dedupe finding on unique_id_from_tool: use the SecHub finding id (value after the last / from the finding's ARN)

Could you add some checks in the unit tests that show the new data in the findings?
@damiencarol Bonjour Damien, pourrais-tu elaborer sur la requete consistant a modifier les unittests pour le filtre d'ingestion des donnees provenant de AWS SecurityHub ?
The parser does not change much in functionality, the big change in this PR is the fact that the deduplication and filtering was not working properly when ingesting one report at a time from AWS SecurityHub.
The behavior observed prior to this change request was as follows:

When drilling down into to a finding , the "imilar findings" section showed all the other AWS Security Hub findings regardless  of the AWS resource associated with the finding.
Each new imported finding marked the previous one as duplicate, which is consistent with the behavior reported in the first bullet point.
In conclusion, there isn't any new data in the findings, the proposed change implements the behavior we come to expect with DefectDojo when it comes to aggregating similar findings, managing duplicate findings and closing stale/old findings.
Feel free to DM me on the OWASP defectodjo Slack channel (handle: Nicolas Landais) to discuss further.
-Merci

@nlandais there is something out of scope in the PR. it seems that one sub-module for documentation is updated.

@valentijnscholten could you take another look, @nlandais made a lot of modifications since the first reviews.


Prepare for May relay

We have a few things we need to take care of in advance of the May relay:

[ ] TRAPI 1.1 deployment (#249)
[ ] Address documentation requirements in README and/or wiki (with links from README)
[ ] Separate document (in wiki?) on testing ("An automated testing plan to verify service availability and basic knowledge retrieval has been developed for each Translator tool, with detailed documentation on the expected results of each test.")

CAM KP tool page needs a revamp, populated with content according to this template.
Note in particular the bulleted items in the description section. These are required per NCATS’ 2021 program milestones.


benchmark: introduce

I would like to know the cost of loader. So to know that I would like to benchmark.
i propose to use node-bench.

bench-node :P


Fix spacing so CLI samples render properly

#8273

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.11 Features: set Milestone to 1.11 and Base Branch to release-1.11
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

/assign @chenopis

/assign @heckj

👍
Going to take me a bit to get used to how to preview/lookup this new location stuff. Pickier about the MD format in interesting new ways, eh? Thanks @tallt0m !!

/approve
/lgtm


Clarify ibmclod_api_key usage

Description
Make it clear that the ibmcloud_api_key input to this module does not have to be the same key used for the IBM Cloud Provider in the provider block. It can be a more restricted key with limited ability to write Log Analysis records to the Cloud Object Storage buckets.
To avoid a breaking change and work for all consumers to change a variable name, ibmcloud_api_key has been retained for internal use inside the module.
The observability_achive example has been updated to show how a second less privileged api key should be passed into the module.
#205
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[x] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Clarify that the ibmcloud_api_key can be a more restrictive key than the key used to configure the IBM Cloud Provider to create the instances.
Updates to observability_archive to show how this key separation works.

Checklist for reviewers

[x] If relevant, a test for the change is included or updated with this PR.
[x] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

:tada: This PR is included in version 2.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


refresh.

@mimig1

Hi @h0n, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (hawong). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@h0n : Thanks for your contribution to the Azure documentation! The author, @h0n, has been notified to review your proposed change.

@h0n - you made this in the public repo, so I think it'll get auto closed. Please resubmit in the private repo.

@mimig1 @h0n Han can we close this?

@jenniehubbard Yes, I will port it to private tomorrow.

@h0n - Thanks for your contribution to the Azure docs. On November 16th, we announced the availability of our Azure technical documentation in a new location on docs.microsoft.com. You can read about it on our team blog. This change means the article source lives in a new GitHub repository. Because there are some changes in the source markdown as a result of the move, we can’t pull your contribution over to the new repo without creating a ton of merge conflicts. We invite you to resubmit the change (if you are willing) to the new version of the article at https://docs.microsoft.com/en-us/azure/documentdb/documentdb-use-cases. Thanks for your understanding.

#please-close The doc has been updated in the new site - @h0n is fine with this.


LCircleMarker with LTtooltip error

Thank you so much for your work
We have faced this problem, when using l-circle-marker with a nested l-tooltip

Error

link to reproduce
Current solution to the problem: set :animate="false"

Hi, thanks for the information. I'll look into it during the week. Glad to hear that you have a temporary workaround.

Hi, I took some time to look into your code. Unfortunately there is no perfect solution for this problem. But let's start from the beginning. This library is just a wrapper around lefalet.markercluster which extends the leaflet functionality. Leaflet.markercluster relies on the internal leaflet function _setPos, which enables them to move the marker away from the original lat/lng coordinate without touching them. This is important, because if you spiderfy the cluster you need to go back to the original values.
Unfortunately even if leaflet called the item circleMarker it has nothing to do with the leaflet marker. The circle marker is a path (leaflet marker are dom objects) and doesn't provide the option to move the drawn marker independent from the original lat/lng values. This behavior leads to the point that the circle marker can't be used within the markercluster with animation="true", because leaflet doesn't provide any way to move them around freely.
With other words. We would need to extend the circleMarker in the leaflet repository to animate circleMarkers within the markercluster.
But there are other solutions you might could use. First one you already found out. Set animate to false, would disable the movement of the markers and the part of the code which relies on the implementation of the _setPos function.
Another one would be to use a custom icon for the marker which displays the circle as an img/svg. This would allow you to use the leaflet marker with a circle shape.
I hope I could give you some insight how leaflet work and what is the root cause of the problem. Might be even worth of a note for the readme. So other people don't run into this issue.

Thanks for the detailed answer!
We are migrating the project from vue 2. There were no problems in the old project (I'll prepare a demo later)
Also, w'll try to consider workarounds.
Thank you!

Okay this would be helpful and interesting to see. Maybe there is a way to work around it, I didn't think of it.

Hey, did you had a chance to create a demo?

Hey, did you had a chance to create a demo?

https://stackblitz.com/edit/vue2-vue-cli-yvpf3b?file=src%2FApp.vue

Thanks. This helped me to verify the source of the problem. In the  vue2 markercluster plugin the default value for       animateAddingMarkers is false. In this project our default value is set to true. If we change these settings we will receive the same result for both versions (vue 2 and vue 3).
But you are right we should use the correct default option here. I'll address this in the upcoming release. Thanks for the information.
Nevertheless for  :animateAddingMarkers="true" the problem will stay the same as explained above.

Default behavior was adjusted in v0.3.0


runtime: TestMutexFairness failures after 120s on openbsd-arm64-jsing with -cpu=10

##### sync -cpu=10
--- FAIL: TestMutexFairness (119.29s)
    mutex_test.go:227: can't acquire Mutex in 10 seconds
FAIL
FAIL	sync	360.801s

greplogs -l -e 'FAIL: TestMutexFairness' --since=2022-02-01
2022-05-26T14:42:13-581a63c/openbsd-arm64-jsing
2022-05-18T00:45:20-1c6706c/openbsd-arm64-jsing
(attn @4a6f656c @golang/openbsd; CC @golang/runtime)
What seems especially odd to me is that the test took almost exactly two minutes to time out both times, even though it should be failing after only 10 seconds:
https://cs.opensource.google/go/go/+/master:src/sync/mutex_test.go;l=226;drc=645d07819b2224ba4d759829443f7c6442162c69
That suggests that either the main TestMutexFairness goroutine isn't being scheduled, or the runtime timer heap isn't being processed.
And in both cases the overall running time of the test is also almost an exact multiple of 120 seconds.
It looks to me like something is causing goroutine scheduling (or perhaps OS thread scheduling?) to wedge until some 2-minute watchdog timer fires. #52703 may be related.

There appears to be a kevent related hang that can be triggered on OpenBSD 7.1 - I've applied a workaround (per #52703)

Also, FWIW the 120s timeout is likely coming from here:
https://build.golang.org/log/afeae56f967155165fedf1eb30224754ebb9b18a


video recording loses audio upon export

hi,
we are seeing a problem in iOS8 where the recorded video, when exported to camera roll using standard code is losing audio - the video is recorded using GPUMovieWriter as mp4 using all standard settings - no change is made to tracks or composition or doing anything to the video - we are not even using avexportsession - simply using ALAssetsLibrary writeVideoAtPathToSavedPhotosAlbum

loses audio upon export to camera roll
it plays back with audio in our app using MPMoviePlayerController
it continues to work in iOS7
further, when we export to desktop and load in quicktime, it doesn't have audio track
but if we open with VLC, it shows audio track and plays audio.

so it seems something changed with iOS8, with apple mp4 video audio format/composition/encoding/playback that gpumoviewriter needs to account for.

is anyone else having this issue?

thx.

Yes we are having the same issue. Audio Export fails in iOS 8, but works fine in iOS 7. Could you fix the problem on your end ?

we face same thing, iOS9 no problem, but iOS8 sometimes lost audio


test: add one more test for mainnet rewards file with big interval

Add supplementary test with rewards file from tx
https://etherscan.io/tx/0x8ee9f4ad6944b45758523fe59b79352fd3f497aee5089ad16bb4355f49a002d5

@lean-apple I am giving you my crown for biggest PR


No native dependencies is not true

The README mentions the package has no native dependencies, yet on macOS, brew install mango && brew install cairo is needed

Seems to have worked on Windows 10 without needing anything extra 🤔

hi, yeah I agree the current phrasing is a bit misleading, as @whyboris noted, they key point is that you don't need to compile any native code in most cases (with the exception of rare OS/NodeJS combinations which don't have pre-built binaries)
I'm open to rewording the strapline, any suggestions?

Just remove the claim or add note that you use node canvas which might need compilation. I for example am not able to use pdf-to-img because node canvas compilation fails for my m3 mac.


SDF -> WoT: Add mapping for sdfProduct

Codecov Report

Merging #16 (931bc11) into main (d94bdc0) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #16   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          355       359    +4
=========================================
+ Hits           355       359    +4

Impacted Files
Coverage Δ

converter/sdf_to_wot.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Unit tests fail

Currently, unit tests of the main branch fail:
mvn clean install
...
Results :

Failed tests:
  VerifyTestNGController.doTestRun:69 Unexpected number of fail verdicts. expected:<0> but was:<2>

Tests run: 21, Failures: 1, Errors: 0, Skipped: 2

[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------

The unit test failures are caused by an commit which has been done after revision 0.2 of this test suite.
See https://github.com/opengeospatial/ets-eo-geojson10/commits/main/ for the list of commits.
I propose to postpone the commits which have been done after revision 0.2 and to create revision 1.0 (Production release) without them.
In addition, those commits have never been installed on Beta.
@ghobona Can you please check the new commits and do you agree with my proposal?

New pull request was created for postponed commits: https://github.com/opengeospatial/ets-eo-geojson10/pull/13

The unit test failures can be reproduced using the branch of #13.


Node information sent to Compliance after first audit run are not accurate

Cookbook version
0.6.0
Chef-client version
12.9.38-1
Platform Details
Windows 7 Enterprise
Scenario:
When a Windows node initiates a chef-client run with the audit cookbook, certain information is sent back to Chef Compliance such as hostname/IP and WinRM account. However the hostname/IP field is not filled in and WinRM account name is listed as Administrator even though node is a domain account with a domain administrative account. This prevents reconnection to the node until this information is entered in manually.
Steps to Reproduce:
Run the audit cookbook for the first time on a Win7 client. View the node information result in Chef Compliance after the chef-client run is complete.
Expected Result:
Hostname/IP of machine and account used to run chef-client command remotely through winrm should be sent back to Chef Compliance and listed in the node details on Chef Compliance page.
Actual Result:
Hostname/IP field is empty and account is listed as generic Administrator

Is this because when you use the audit cookbook, its actually using the Chef client keys.
So it is node -> chef server -> chef compliance
node -> chef server is handled by the client.pem file on the machine via HTTPS to Server
chef server -> chef compliance is handled by the integration between server & compliance.
I wouldn't expect the username and password that Chef Client ran as to populate in Compliance, however it should provide the ability to scan / provide information to scan from within compliance.

The ability to scan/provide information to scan from within Compliance is not there because the parameters used to connect to the node to winrm are not filled in properly.
Even disregarding the username/password, the hostname/ip should populate in Compliance which it is not doing either.

@chef09210 Thanks for reporting. We are aware, that this is not working as expected. This will be fixed within one of the next releases.

@chef09210 We refactored the audit cookbook to make it easier to maintain and improve the quality. Is that issues still persistent with the version 2.0 of the audit cookbook.
We retrieve the information from Chef
https://github.com/chef-cookbooks/audit/blob/master/files/default/audit_report.rb#L105-L116

@chef09210 I am going to close this issue. Please reopen if the issue persists with audit cookbook version 2


feat(julials): add JuliaActivateEnv to activate a julia environment

This adds a command for the additional julials API for activating an environment. This is based on how extra functionality is added into the texlab configuration

Looks like this fails the CI/CD check regarding the use of dirname and it potentially being used in the root_dir. I'm just confirming that the use of dirname does not go against the directive listed in the CI/CD check :)


Matrix Subtraction in C

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Matrix Subtraction in C
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #5580

Respected Mentors please review my PR

@HarshCasper I have made all changes please review it once

@ankitaggarwal23 made changes please check

@ankitaggarwal23 please check


File: Added DevOps Resources

Resolves issue #7199
Added DevOps and Golang resources
What does this PR do?
Add resource(s)
For resources
Description
I have added DevOps and Golang video tutorials available on youtube
Why is this valuable (or not)?
As we know DevOps is an emerging field, and this is really valuable.
How do we know it's really free?
These resources are in video format and available on youtube
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Hey, Please help me with that. I don't know why I'm failing one progress check. I managed all resources to be in alphabetical order.

Hey, Please help me with that. I don't know why I'm failing one progress check. I managed all resources to be in alphabetical order.

I don't know, Why this is failing all the time, Please help me with it. I already managed to arrange all those resources in alphabetical order.


iphlpapi.dll not found

Hi,
What version of Zeroconf were you using on what platform?

[mono] Unhandled Exception:
[mono] System.DllNotFoundException: iphlpapi.dll
[mono]   at (wrapper managed-to-native) Zeroconf.NetworkInterface:GetBestInterface (uint,uint&)
[mono]   at Zeroconf.NetworkInterface+<NetworkRequestAsync>d__0.MoveNext () [0x000f5] in C:\Users\MXS57\Source\Workspaces\iComfort3Final\Lennox.iC3.Mobile.Common\Zeroconf.Shared\NetworkInterface.cs:62
[mono] --- End of stack trace from previous location where exception was thrown ---
[mono]   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in /Users/builder/data/lanes/2970/46c3f7e0/source/mono/external/referencesource/mscorlib/system/runtime/exceptionservices/exceptionservicescommon.cs:143
[mono]   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x00047] in /Users/builder/data/lanes/2970/46c3f7e0/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:201
[mono]   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x0002e] in /Users/builder/data/lanes/2970/46c3f7e0/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:170
[mono]   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x0000b] in /Users/builder/data/lanes/2970/46c3f7e0/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:142
[mono]   at System.Runtime.CompilerServices.ConfiguredTaskAwaitable+ConfiguredTaskAwaiter.GetResult () [0x00000] in /Users/builder/data/lanes/2970/46c3f7e0/source/mono/external/referencesource/mscorlib/system/runtime/compilerservices/TaskAwaiter.cs:477
[mono]   at Zeroconf.ZeroconfResolver+<ResolveInternal>d__10.MoveNext () [0x001cd] in C:\Users\MXS57\Source\Workspaces\iComfort3Final\Lennox.iC3.Mobile.Common\Zeroconf.Shared\ZeroconfResolver.cs:284

I wish to change one login from your plugin. I just downloaded the source code.
it executed successfully. But thrown exception DLL not found.

The library does not currently support mono/Linux as you can see a native call is required. I'm happy to entertain a pull request if you can recreate the logic that we get from iphlpapi.dll The other issue is that NuGet does not have a mono-specific TFM, so we cannot special-case mono in the NuGet package.

The latest 2.7.0 code does not use that p/invoke code. I'm closing this out, please open a new issue if needed.
Thanks!


Update Service_Fabric_ReleaseNotes_80CU2.md

Remove reference to 1809 deprecation as support ended 5/11/2021

#sign-off


Bad request when attempting to access settings/config

Working as designed — the admin UI is restricted to the corporate network.

Ah, VPN must've dropped, ty sir


Rate limiting

Looks fine initially, if we need to tweak it later on we can

Looks fine initially, if we need to tweak it later on we can

I think we should just use this actually: https://github.com/benwis/tower-governor


[AKS] Add option Windows2019, Windows2022 to --os-sku for az aks nodepool add

[AKS] Add option Windows2019, Windows2022 to --os-sku for az aks nodepool add
Migrate from Azure CLI extension aks-preview: https://github.com/Azure/azure-cli-extensions/pull/4549
Related command

az aks nodepool add --os-type Windows --os-sku Windows2019
az aks nodepool add --os-type Windows --os-sku Windows2019
Description

Customers can use this option to select OSSKU when creating Windows node pools.
Testing Guide

azdev test --live test_aks_nodepool_add_with_ossku_windows2022
History Notes
N/A

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Hi @AbelHu @FumingZhang , could you help to review this PR? Thank you.

AKS

Hi @zhoxing-ms Could you help to merge this PR? Thank you.


max-width to contain all images by default

Addresses #454
Decided to make sure max-width: 100% is applied to all iframes as well.

:+1: - chances of an image or iframe wanting to be more than the container/screen are so rare that this blanket style is a win.


feat(create-vite): set "strict: true" in tsconfig.node.json

Description
When using @typescript-eslint/strict-type-checked rule set on a fresh Vite project and vite.config.ts is included in the files that eslint runs on, it results in the following error:
  0:1  error  This rule requires the `strictNullChecks` compiler option to be turned on to function correctly  @typescript-eslint/prefer-nullish-coalescing
  0:1  error  This rule requires the `strictNullChecks` compiler option to be turned on to function correctly  @typescript-eslint/no-unnecessary-condition

Given that we have strict enabled in the main tsconfig.json it makes sense to have it enabled here too.
Here's an example eslint.cjs to use in a fresh Vite project where you can see this error:
module.exports = {
  root: true,
  env: { browser: true, es2020: true },
  extends: [
    "eslint:recommended",
    "plugin:@typescript-eslint/strict-type-checked",
    "plugin:@typescript-eslint/stylistic-type-checked",
  ],
  parser: "@typescript-eslint/parser",
  parserOptions: {
    project: ["./tsconfig.json", "./tsconfig.node.json"],
    tsconfigRootDir: __dirname,
  },
};

What is the purpose of this pull request?

[ ] Bug fix
[X] New Feature
[ ] Documentation update
[ ] Other

Thanks for merging. @patak-dev do you think its worth me opening a separate PR to also add the following three items from tsconfig.json to tsconfig.node.json for parity?
    /* Linting */
    "strict": true,
+   "noUnusedLocals": true,
+   "noUnusedParameters": true,
+   "noFallthroughCasesInSwitch": true,

If so, are there any other additional compiler options I should copy across at the same time? I did read the documentation on references but its not clear to me which compiler options, if any, are inherited from tsconfig.json into tsconfig.node.json.
Clearly strict isn't inherited as I wouldn't have got the error using @typescript-eslint/strict-type-checked otherwise which caused me to open this PR in the first place, so I'm guessing nothing is inherited and all compiler options need to be explicitly defined in tsconfig.node.json?


cant parse hebrew

Hi, I played with the tel aviv net and the software crashed, this message was in the output file
[2023-11-24T08:46:41Z WARN  usvg_text_layout] No fonts with a ם/U+5DD character were found.

I think this is the right font
https://fonts.google.com/noto/specimen/Noto+Sans+Hebrew

Thanks! I hope to have some time to catch up on A/B Street issues in the next few weeks

Road names are showing up now! If there's any rendering issues, let me know (though I'm near powerless to fix them -- relying on usvg and some other lower-level libraries for all the text rendering).
I'll keep looking into the geometry crash; that's an ongoing problem everywhere in this codebase

(There will be a new release at some point, but in the meantime, the LTN tool is up-to-date here: https://a-b-street.github.io/abstreet/web/ltn.html?system/il/tel_aviv/maps/center.bin)


PairExtraordinaire x2 Bronze

PairExtraordinaire x2 Bronze
Co-Authored-By: Yolwoocle<EMAIL_ADDRESS>Co-Authored-By: PouletRaPHOO<EMAIL_ADDRESS>Co-Authored-By: Linus Torvalds<EMAIL_ADDRESS>Co-Authored-By: utopy<EMAIL_ADDRESS>Co-Authored-By: Theobosse<EMAIL_ADDRESS>Co-Authored-By: Gouspourd<EMAIL_ADDRESS>Co-Authored-By: Maxio<EMAIL_ADDRESS>Co-Authored-By: Notgoyome<EMAIL_ADDRESS>Co-Authored-By: Antoine Roucau<EMAIL_ADDRESS>Co-Authored-By: PoulpitoGDL<EMAIL_ADDRESS>

Y'all did this just for the achievement. 😕📸

Y'all did this just for the achievement. confusedcamera_flash

:) Maybe


Design a loader/spinner for the infowindow cover

We have a loader for the general content of the infowindow, but I think we should have one for the cover images.

I'd use the same one

the actual proposed one is too subtle IMO

maybe we could improve the contrast.

There has been no activity on this issue for several months and that's why we are closing it (automatically). If you think this still needs to be addressed please reopen it. The good news is that we are now working on the next version of CARTO.js, stay tuned.


"New [School/Department/Course/Section" not keyboard accessible

In the "Schools" view, the "New School" link cannot be accessed using only the keyboard.

mine


[SPARK-32469][SQL] ApplyColumnarRulesAndInsertTransitions should be idempotent

What changes were proposed in this pull request?

This PR makes ApplyColumnarRulesAndInsertTransitions idempotent (assuming the custom columnar rules are also idempotent).
Why are the changes needed?

It's good hygiene to keep rules idempotent
Does this PR introduce any user-facing change?

No
How was this patch tested?

new suite

cc @andygrove @tgravescs @maryannxue

Test build #126727 has started for PR 29273 at commit 50d46d9.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31350/
Test PASSed.

Test build #126727 has finished for PR 29273 at commit 50d46d9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126727/
Test PASSed.

Test build #126982 has started for PR 29273 at commit a1b5664.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31606/
Test PASSed.

@revans2

Test build #126982 has finished for PR 29273 at commit a1b5664.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class RowToColumnarExec(child: SparkPlan) extends RowToColumnarTransition

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126982/
Test PASSed.

thanks for review, merging to master!


Unable to change password, role Unknown after logout (v2)

I am unable to change other users password. No error reported on Save. Can not log in with new password.
After checking the database the update date column still shows the old date (no update).
After logging out then loggin in I loose my admin rights and I become role "Unknown".
Restarting the container solves this issue and I am administrator once again.
This is the container log after server start:
$ node server.js
Listening on port 3000 url: http://28344e9f25e6:3000
PrismaClientValidationError:
Unknown field `teamUser` for include statement on model Team. Did you mean `teamUsers`?
    at ti.validate (/app/node_modules/@prisma/client/runtime/library.js:149:91)
    at Wr.createMessage (/app/node_modules/@prisma/client/runtime/library.js:164:13655)
    at /app/node_modules/@prisma/client/runtime/library.js:174:10933
    at runInChildSpan (/app/node_modules/@prisma/client/runtime/library.js:70:25817)
    at t._executeRequest (/app/node_modules/@prisma/client/runtime/library.js:174:10922) {
  clientVersion: '4.11.0'

I've ran the migration script according to instructions from here https://github.com/umami-software/umami/issues/1887 by berti92.

Theres an issue in the prisma schema file for MySQL causing the temaUser error, which I assume you are using #1893? I'm not sure it's related to the password issue, so I'm going to look into that now.