yarp: 2.3.68 -> <IP_ADDRESS>

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpidl_rosmsg -h got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpidl_rosmsg --help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpidl_rosmsg help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpserver --help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarp help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarp version and found version <IP_ADDRESS>
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarp-config --help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarp-config --version and found version <IP_ADDRESS>
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpdev -h got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpdev --help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpdev help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpmanager-console --help got 0 exit code
ran /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin/yarpdatadumper --help got 0 exit code
found <IP_ADDRESS> with grep in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>

cc @nico202 for review

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘yarp-<IP_ADDRESS>’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/applications/science/robotics/yarp/default.nix:30 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libyarpmod.so
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_gsl.so.<IP_ADDRESS>
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libyarpc.so
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_dev.so.<IP_ADDRESS>
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_init.so.<IP_ADDRESS>
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_name.so.<IP_ADDRESS>
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib  /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin
patching script interpreter paths in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>
checking for references to /tmp/nix-build-yarp-<IP_ADDRESS>.drv-0 in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>...

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/lib/libYARP_init.so.<IP_ADDRESS>
shrinking /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/lib/libYARP_name.so.<IP_ADDRESS>
shrinking /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/lib/libYARP_dev.so.<IP_ADDRESS>
shrinking /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/lib/libYARP_gsl.so.<IP_ADDRESS>
shrinking /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/lib/libYARP_sig.so.<IP_ADDRESS>
shrinking /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/lib/libYARP_OS.so.<IP_ADDRESS>
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/lib  /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>/bin
patching script interpreter paths in /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>
checking for references to /build in /nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>...

Uhm I've not been using it for a while, if anybody want to maintain it please do..

@GrahamcOfBorg build yarp
Let's test a new feature ...

Success on aarch64-linux (full log)
Partial log (click to expand)
/nix/store/z1bmgclim7g2srzbrqv8j1xgdlkr2xmh-yarp-<IP_ADDRESS>

No attempt on x86_64-darwin
Skipped because they don't instantiate on this system: yarp
No log is available.

Success on x86_64-linux (full log)
Attempted: yarp
Partial log (click to expand)
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_name.so.<IP_ADDRESS>
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_OS.so.<IP_ADDRESS>
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libyarpc.so
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_dev.so.<IP_ADDRESS>
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_sig.so.<IP_ADDRESS>
shrinking /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib/libYARP_gsl.so.<IP_ADDRESS>
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/lib  /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>/bin
patching script interpreter paths in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>
checking for references to /tmp/nix-build-yarp-<IP_ADDRESS>.drv-0 in /nix/store/bp68k0rdxw13rxgy93lqlqmrmprz272g-yarp-<IP_ADDRESS>...


Check Password for unconfirmed user.

Describe the bug
when user is unconfirmed the following fails with "Error: Error Occured in controller:  with exception User is not confirmed."
var passValid = await ((CognitoUserManager<CognitoUser>)_userManager).CheckPasswordAsync(findUser, model.Password).ConfigureAwait(false);
The idea is to check if password is correct for unconfirmed user without logging them in but to redirect them to Resend Email Confirmation page. I cannot just allow anyone resend confirmation by providing a valid registered email. Similarly once the user has signed up but decided for whatever reason to clear their localstorage or use another browser they will never be able to confirm their email. Is there some way around this ?
Expected Behavior
var passValid = await ((CognitoUserManager<CognitoUser>)_userManager).CheckPasswordAsync(findUser, model.Password).ConfigureAwait(false);
Either works for unconfirmed users or there is another way to verify that password is correct without logging a user in
Current Behavior
Error: Error Occured in controller:  with exception User is not confirmed.
Reproduction Steps
var passValid = await ((CognitoUserManager<CognitoUser>)_userManager).CheckPasswordAsync(CognitoUser, model.Password).ConfigureAwait(false);
OR
var passValid = await _userManager.CheckPasswordAsync(CognitoUser, model.Password).ConfigureAwait(false);
for unconfirmed user fails
Possible Solution
No response
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
Amazon.AspNetCore.Identity.Cognito 3.0.1
Targeted .NET Platform
.NET 6.0
Operating System and version
Windows 10

Hi @Philipp15,
Thank you for reporting the problem. I was able to reproduce the exception ‘UserNotConfirmedException’ on a sample ASP.NET MVC web application.
If the user exists in Cognito and the password is correct, then this exception is thrown.
You can handle this scenario in couple of ways.

You can use the AdminGetUserAsync() to check if the user status type is ‘Unconfirmed’. If so, then provided the supplied username and password are valid, you can resend the confirmation code to user by ResendConfirmationCodeAsync() and redirect the view to ‘Email confirmation’ page.

                var user = await _userManager.FindByNameAsync(Input.UserName);

                AmazonCognitoIdentityProviderClient clientobj= new AmazonCognitoIdentityProviderClient();

                AdminGetUserRequest userRequest = new AdminGetUserRequest
                {
                    Username = Input.UserName,
                    UserPoolId = "******",
                };

                var response = await clientobj.AdminGetUserAsync(userRequest);

                if (response.UserStatus== UserStatusType.UNCONFIRMED)
                {
                    var codeRequest = new ResendConfirmationCodeRequest
                    {
                        ClientId = "******",
                        Username = Input.UserName,
                    };

                    var coderesponse = await clientobj.ResendConfirmationCodeAsync(codeRequest);
                    return RedirectToPage("./ConfirmEmail");
                }

Since this exception is thrown when the user exists in Cognito and the password is correct (entered while registration), you can catch the exception (UserNotConfirmedException) and redirect to the ‘Email Confirmation’ page after resending the confirmation code.

Please let me know if this would help resolve your problem or else kindly correct me.
Regards,
Chaitanya

Hi Chaitanya,
Thanks for your reply.

If I follow the point 1 route ... that means that anyone with wrong password can force send a confirmation email to account they do not own ...I believe that`s not so good from security perspective

Regarding your second point "For me when I run the following code

var result = await _signInManager.PasswordSignInAsync(model.Email,                  model.Password, true, false).ConfigureAwait(false);
for unconfirmed user no exception is thrown ... instead the result I get is just "result.Succeeded == false" with no exception
Full Code Segment:
try
               {
                   var result = await _signInManager.PasswordSignInAsync(model.Email,
                model.Password, true, false).ConfigureAwait(false);

                   if (result.Succeeded)
                   {
                       // some more code if succeeded
                   }

               }

               catch (UserNotConfirmedException unce)
               {
                   _logger.LogError(unce.Message); // never hits this if password is correct
               }
               catch(Exception ex)
               {
                   _logger.LogError(ex.Message); // never hits this if password is incorrect either
               }

Hello @Philipp15,
Thank you for the response.
Approach 1 that I proposed was based on the fact that correct username and password is provided by the user and you get the exception. You shouldn’t be resending the confirmation code unless the username and password matches and the user status is ‘Unconfirmed’. Apologies, for the confusion.
When I execute your sample code, I am getting UserNotConfirmedException, provided I pass the username and the correct password.
public virtual async Task<SignInResult> PasswordSignInAsync(string userName, string password, bool isPersistent, bool lockoutOnFailure)

If you trying to use above method from Microsoft.AspNetCore.Identity, you are passing Model.Email as string whereas the method might be expecting a username. Hence you aren’t getting the exception.
PasswordSignInAsync() from Microsoft.AspNetCore.Identity.dll:
        public virtual async Task<SignInResult> PasswordSignInAsync(string userName, string password,
            bool isPersistent, bool lockoutOnFailure)
        {
            var user = await UserManager.FindByNameAsync(userName);
            if (user == null)
            {
                return SignInResult.Failed;
            }

            return await PasswordSignInAsync(user, password, isPersistent, lockoutOnFailure);
        }

Please let me know for further questions.
Regards,
Chaitanya

Hi Chaitanya,
I can confirm that when I changed email to username the UserNotConfirmedException was thrown on correct password, thank you for your help and swift reply!!


docs: API docs enhancements

[ ] Document symbol in every package that has it, eg: @libp2p/connection
[ ] "Defined in " is missing packages/ for re-exported Libp2p interface in libp2p package

### Tasks
- [ ] https://github.com/libp2p/js-libp2p/issues/1860
- [ ] https://github.com/libp2p/js-libp2p/issues/1913
- [ ] Remove reference to `stat` object in `Connection` docs
- [ ] Ensure that well-known `libp2p.services` are documented
- [ ] https://github.com/libp2p/js-libp2p/issues/2096

first bullet point related https://github.com/ipfs/eslint-config-ipfs/pull/190, needs to be pulled thru aegir and enforced here

@wemeetagain I have moved the other issue to https://github.com/libp2p/js-libp2p/issues/2096 and we can use this issue to track other API Doc enhancements


List of urlParams ?

Hello,
where might I find a list of the urlParams mentioned in the docs?
e.g.:
client.get(path, urlParams);

Thank you

Usually, Twitter responds with an error and a list of valid params.
I created the request using postman first: https://documenter.getpostman.com/view/9956214/T1LMiT5U
However, while this url is a valid request:
https://api.twitter.com/2/tweets/TWEET_ID?tweet.fields=author_id,conversation_id,referenced_tweets
both of these arent:
const data = client.get("tweets/TWEET_ID", {
  "tweet.fields": " author_id,conversation_id,referenced_tweets",
});

const data = client.get("tweets", {
  ids: "TWEET_ID",
  "tweet.fields": " author_id,conversation_id,referenced_tweets",
});

The url params used by the client methods get, post, and stream all are the same as those used directly by Twitter (see examples/get_tweets.js for an example) they can be encoded two ways:
{
  'foo.bar': 100,
  baz: 200,
}

would result in ?foo.bar=100&baz=200
while so would the following:
{
  foo: { bar: 100 },
  baz: 200,
}


Ok 5 report reviews

Jira Link:
[Jira: ](https://champlainsaintlambert.atlassian.net/jira/software/projects/OK/boards/55?selectedIssue=OK-5)
Context
Users can now report content
Changes

added report popup
report popup called when clicking report
added firebase function to check if user already reported a review
added firebase function to add report
added report type

Before and After UI (Required for UI-Impacting PRs)
After:

Everything looks good, can you just explain what the change to the test had to be?

Yup! The problem was the test assumes there is always a dropdown button even when logged out however this does not make sense. Someone who is not logged in should not be able to report a review because this could be exploited and spammed. Therefore I removed the dropdown button when a user is not logged in which means that cypress should not be able to see it. I changed the test so that it checks to make sure that there is no drop-down button when logged out.
:shipit:


make lint fails due to missing documentation

Describe the change
Running make lint on the main branch yields the following errors:
error: missing documentation for the crate
  --> testing/ef-tests/tests/tests.rs:1:1
   |
1  | / #![cfg(feature = "ef-tests")]
2  | |
3  | | use ef_tests::{cases::blockchain_test::BlockchainTests, suite::Suite};
...  |
76 | |     general_state_test!(vm_tests, VMTests);
77 | | }
   | |_^
   |
   = note: `-D missing-docs` implied by `-D warnings`
   = help: to override `-D warnings` add `#[allow(missing_docs)]`

error: could not compile `ef-tests` (test "tests") due to 1 previous error
warning: build failed, waiting for other jobs to finish...
error: missing documentation for the crate
 --> crates/rpc/rpc-testing-util/tests/it/main.rs:1:1
  |
1 | / mod trace;
2 | |
3 | | const fn main() {}
  | |__________________^
  |
  = note: `-D missing-docs` implied by `-D warnings`
  = help: to override `-D warnings` add `#[allow(missing_docs)]`

error: could not compile `reth-rpc-api-testing-util` (test "it") due to 1 previous error
error: missing documentation for the crate
 --> crates/net/network/tests/it/main.rs:1:1
  |
1 | / mod big_pooled_txs_req;
2 | | mod connect;
3 | | mod multiplex;
4 | | mod requests;
... |
8 | |
9 | | const fn main() {}
  | |__________________^
  |
  = note: `-D missing-docs` implied by `-D warnings`
  = help: to override `-D warnings` add `#[allow(missing_docs)]`

We should either add docs to these crates or add #[allow(missing_docs)] to pass lint step
Additional context
No response

Seems there are several other places missing documentation other than the ones noted above that would cause lint to fail. This looks like a good first issue for a new contributor

@emnul can I take a look at this?

@emnul can I take a look at this?

Go for it! Although, I'm not a maintainer so I cannot assign it to you, but you don't need to wait until then to get started.

closing in favor of #10827


support disabling of dojo joining

This adds a joinable tag to dojos, which allows the disabling of new joins to a dojo.

What is the usecase for someone not being able to join a dojo? Official dojos are already not joinable (except as a student for a course dojo), I don't think we even really need to keep requiring community dojos to be joinable, they should just follow the same model as official dojos. Finally there are private dojos, which have a password to gatekeep joining.

But it passed CI.....
Honestly, I'd forgotten about passwords. Do we have an example of a password-protected dojo I could play around with?

@zardus CI is only testing that both parts build and that the website comes up. We need to actually add test cases that exercise functionality so you can get the warm fuzzy of CI
https://github.com/pwncollege/dojo/pull/208#issuecomment-1703698912

It may have passed Continuous Integration, but it hasn't passed Connor Integration 😉
There aren't any open source private dojos, since that wouldn't make any sense (maybe the password shouldn't be specified in the yml?).
https://github.com/pwncollege/cse365-2023-summer/blob/main/dojo.yml is an example of a dojo with a password. It's just password: MYPASSWORD at the top level. I probably wouldn't modify that dojo though...
For any on lookers, don't worry, there aren't any fun private challenges in that dojo.


Twitter Updates

Checklist

[X] I'm reporting a broken site support
[X] I've verified that I'm running youtube-dl version 2021.12.17
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones

Verbose log

REDACTED username ~ % youtube-dl -g https://twitter.com/1500tasvir/status/1577533879283585025 -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-g', 'https://twitter.com/1500tasvir/status/1577533879283585025', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.11.3 (CPython) - macOS-13.4.1-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by <HTTPError 404: 'Not Found'>); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/lib/python3.11/site-packages/youtube_dl/extractor/common.py", line 634, in _request_webpage
    return self._downloader.urlopen(url_or_request)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/youtube_dl/YoutubeDL.py", line 2288, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 525, in open
    response = meth(req, response)
               ^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 634, in http_response
    response = self.parent.error(
               ^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 563, in error
    return self._call_chain(*args)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 496, in _call_chain
    result = func(*args)
             ^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 643, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)

## Grabbing URLs from Twitter not working. I assume due to their new crack-down on web scrapers?

<!--
Provide an explanation of your issue in an arbitrary form. Provide any additional information, suggested solution and as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

Grabbing URLs from Twitter not working. I assume due to their new crack-down on web scrapers?

https://github.com/yt-dlp/yt-dlp/issues/7473 says you now have to authenticate, which I guess means passing cookies from your logged-in browser session.

says you now have to authenticate,
which I guess means passing cookies from your logged-in browser session.

... Thanks to bashonly 👍 , "upstream" do have a plan B 😜 in the shape of https://github.com/yt-dlp/yt-dlp/pull/7476, which, once tested here, delivers what it promises (unlike politicians 😞 ) :
python yt-dlp -vF "https://twitter.com/1500tasvir/status/1577533879283585025" =>

[debug] Command-line config: ['-vF', 'https://twitter.com/1500tasvir/status/1577533879283585025']
[debug] Encodings: locale cp1253, fs utf-8, pref cp1253, out utf-8 (No VT), error utf-8 (No VT), screen utf-8 (No VT)
[debug] yt-dlp version<EMAIL_ADDRESS>[8776349ef] (zip)
[debug] Python 3.7.16 (CPython x86 32bit) - Windows-Vista-6.0.6003-SP2 (OpenSSL 1.1.1s  1 Nov 2022)
[debug] exe versions: none
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1858 extractors
[twitter] Extracting URL: https://twitter.com/1500tasvir/status/1577533879283585025
[twitter]<PHONE_NUMBER>283585025: Downloading syndication JSON
WARNING: [twitter] Not all metadata is available without authentication
[debug] [twitter] Extracting from video info:<PHONE_NUMBER>206241280
[twitter]<PHONE_NUMBER>283585025: Downloading m3u8 information
[debug] Sort order given by extractor: res, br, size, proto
[debug] Formats sorted by: hasvid, ie_pref, res, br, size, proto, lang, quality, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, asr, vext, aext, hasaud, source, id
[info] Available formats for<PHONE_NUMBER>206241280:
ID       EXT RESOLUTION |   FILESIZE  TBR PROTO | VCODEC      ACODEC
-----------------------------------------------------------------------
hls-256  mp4 480x270    | ~520.10KiB 256k m3u8  | avc1.4d001e mp4a.40.2
http-256 mp4 480x270    | ≈520.10KiB 256k https | unknown     unknown
hls-832  mp4 640x360    | ~  1.65MiB 832k m3u8  | avc1.4d001f mp4a.40.2
http-832 mp4 640x360    | ≈  1.65MiB 832k https | unknown     unknown

As bashonly suggests in the yt-dlp PR, we should wait to see how the Muskified Twitter turns out before committing a new extractor.

This will return a JSON file containing the video links:
https://cdn.syndication.twimg.com/tweet-result?id=1577533879283585025

This will return a JSON file containing the video links:
https://cdn.syndication.twimg.com/tweet-result?id=1577533879283585025
doesnt seem to work atm.

doesnt seem to work atm.

Yeah. A token is now required. Try this:
https://cdn.syndication.twimg.com/tweet-result?id=1577533879283585025&token=422kr3t824n

Is that a random value, or is there an algorithm for generating it?
BTW, the current yt-dlp code adds a User-Agent header with value Googlebot. I presume that's supposed to bypass this.

Yeah. A token is now required. Try this:
https://cdn.syndication.twimg.com/tweet-result?id=1577533879283585025&token=422kr3t824n

gotta love when people post crap with no context. by doing that, you diminish the value of what you are posting nearly to the point of it being worthless. so please dont. at any rate, go here and dance the dance:
https://publish.twitter.com
and you get your token. the good side is based on the previous comment, token seems to be long lasting, as least two days. bad side is token seems to be dynamically created with JavaScript by one of these:

https://platform.twitter.com/embed/embed.8601.9934bea25cda9d7ebe46.js
https://platform.twitter.com/embed/embed.Tweet.c66f70f23355733e0f73.js
https://ton.twimg.com/syndication/publish/b568e671052c8/js/publish.js

or you can still use the Twitter Android OAuth flow I coded in my repo.

LOL, just realized the token can be any string:
https://cdn.syndication.twimg.com/tweet-result?id=1577533879283585025&token=!

LOL, just realized the token can be any string:

LOL again. Why didn't I think of that? By the way, I see that you deleted your previous comment. Think before you jump, as they say.

my previous comment still applies. I said comments like the one you posted are NEARLY worthless. which they are. if you have context, you should provide it. otherwise you're just forcing others to try to figure out what you already figured out. and maybe they cant, so then that knowledge is just lost forever if you decide to stop commenting. I have seen it happen many, many times.

@1268
I didn't figure anything out. I got the link from the developer tool after running this link to get the publicly accessible version of the tweet:
https://platform.twitter.com/embed/Tweet.html?id=1577533879283585025
I posted what I found for others to try out. Further investigation would have been premature. As you now know. What did you expect from me? An explanation that I used developer tools to look at a tweet?

What did you expect from me?

what you have now commented:

https://platform.twitter.com/embed/Tweet.html?id=1577533879283585025

if you'll notice, your most recent comment is the first time that URL has been posted on this thread. before that, the  /tweet-result URL was just a mystery URL that came from thin air.

@1268
That pathway is not needed to obtain the video. That's why I didn't mention it. It might be useful for the extraction of other information but I don't know what youtube-dl would want from it.
Looking through the current iteration of the extractor code, I see that most of it is basically junk now due to the changes that Musk has wrought. Getting a tweet video is easy with the pathway that I have given. Broadcasts still work with the guest token (for now). However, Audio Spaces won't work without login cookies. Should just focus on getting youtube-dl working with regular tweets. I've told you what you need to know so what are we arguing about?

I am not arguing. just requesting that people provide a little context instead of dropping magic URLs out of thin air. which you have now done. its much more helpful to the community. if you just drop some magic URL, then as soon as the vendor makes any change the URL usually becomes useless, until the person decides to provide another magic URL and/or solution.

As if we haven't been relying on "magic links" all along. The extractors are full of magic links with no indication whatsoever of how they were uncovered in the first place. The previous magic link was rendered obsolete by Twitter. Now we have another magic link. If that stops working then somebody will find yet another magic link. This will happen without extensive explanation. Yeah, this is what-about-ism but you're complaining about something that has always been true. I don't see how that can change. Nobody ever explains anything in those extractors. That's why I rarely look at them. It's easier to just forge my own path. No, I don't contribute to youtube-dl (except for the occassional comment). I have my own tools.

As if we haven't been relying on "magic links" all along. The extractors are full of magic links with no indication whatsoever of how they were uncovered in the first place.

I cant speak for YouTube-DL code, but for my own, I try to always document any magic values that I use.

The previous magic link was rendered obsolete by Twitter.

I dont care about previous links, I care about the original comment you made with the magic link. please stay on topic.

Now we have another magic link.

right, but as you've demonstrated, you always had context for it, you just only provided it upon request. I am asking that you just provide it up front going forward, so that people dont have to ask for it. if you just randomly found it online thats fine, but if you actually have context about how you got the link (which you did), then its very helpful to provide that.

If that stops working then somebody will find yet another magic link.

not always. if a user provides context, then sometimes you can use the method they used to deal with any vendor changes. but if user doesn't provide that context, then it makes it that much more difficult.

Yeah, this is what-about-ism but you're complaining about something that has always been true.

again I dont care about history, I care about YOUR ACTIONS. you have the context, and you have the choice to post it or not. you chose not to. I am only asking going forward that if you have the context, provide it.

I don't see how that can change.

see above.

Nobody ever explains anything in those extractors.

thats a choice of each person writing code and providing help. just because other people choose to omit context, doesn't mean you have to do the same. you can make your own choices.

Are you on a mission? That's fine. I hope that this discussion proves useful to the community. Are you doing anything with the information that I've provided? Since nobody else is offering a solution then maybe I deserve a bit of thanks instead of a thrashing. LOL.

Are you on a mission?

its a simple request to provide context, when you have it. you can either honor the request or ignore it, up to you.

Are you doing anything with the information that I've provided? Since nobody else is offering a solution then maybe I deserve a bit of thanks instead of a thrashing.

I reverse the Android client a month ago:
https://github.com/1268/media/tree/v1.5.6/twitter
however it seems since the change from Twitter to X, a sign in is required, even on the Android app. so my method no longer works. but to be fair, I did discover the token can be hard coded in this thread, so I think thats worth something. I will probably implement your method as a replacement for my own, so thanks for that.

You could try doing what I do. Make an account on Twitter. All you need is an email (use a throwaway). Log in to the account and never log out. Extract the cookies called "ct0" and "auth_token". Both cookies need to be sent along with the usual Bearer token. The request must also contain a header called "x-csrf-token" set to the exact same value as the "ct0" cookie. For prove that this works:
https://8chananon.github.io/app/launch-8kun_media.htm?tweet##1695105179593658703
Of course, it is possible that somebody might steal your cookies and mess around with the account. In which case, you can either make a new one or have an option for users to provide their own cookies.

I'm back-porting the current yt-dlp Twitter/X extractor ATM. I suggest anyone who wants to contribute should hold back until this becomes a PR and then propose any improvements.

i can confirm the 404  youtube-dl version 2021.12.17

I get the same 404 error. I can also confirm that the method outlined by @8chanAnon above works to get the metadata JSON file.


Improvements In streamlit_web_interface.py & README.md

Kindly view the commits

Closing this as #7 overlaps.


Tomcat启动dubbo方式监控问题

Please answer these questions before submitting your issue.

Why do you submit this issue?
[] Question or discussion

Question

What do you want to know?
我们目前有个项目是这样的结构：Web端是tomcat 8.x+Spring MVC 4.x，服务端是tomcat8.x+dubbo 2.8.4，我按照入门文档部署了skywalking 3.2.6的单机版，可以实现监控了。但是存在两个问题：
1）在拓扑图中Web端链路和服务端链路是分开的，即两个分支，能否合并成一个分支，是否需要配置dubbo monitor？
2）JVM和CPU等信息在UI中没有监控，Dashboard中的Menu菜单中没有这项监控。
谢谢skywalking team.

Please rewrite the issue description in English for helping global community to find existed issue.

We currently have a project like this: the Web tier is tomcat 8.5.x+spring mvc 4.3.x, and the Server tier is tomcat 8.5.x+dubbo 2.8.4, and I deploy the single version of skywalking 3.2.6 according to the quick start, which can be monitored. But there are two problems:

In topology, the web trace and the server trace are separated, that is, two branches, can it be merged into one branch, and I've deployed the dubbo monitor.
The information such as JVM and CPU is not monitored in UI, and there isn't JVM monitoring item in the Menu in Dashboard.

@weenyaimac Updated your issue, and removed your latest comments.

In topology, the web trace and the server trace are separated, that is, two branches, can it be merged into one branch, and I've deployed the dubbo monitor.

Are you active com.alibaba.dubbo.monitor.support.MonitorFilter filter in Dubbo? Our instrumentation based on that.

The information such as JVM and CPU is not monitored in UI, and there isn't JVM monitoring item in the Menu in Dashboard.

Which version do you use for tests? If you have any data, JVM definitely metrics are definite included.

@wu-sheng Thanks first. I remember MonitorFilter in Dubbo is activeted by default. Pls see following dubbo code:
@Activate(group = { **Constants.PROVIDER, Constants.CONSUMER** })
public class MonitorFilter implements Filter {}
I just solved the first question, in the way of which I deleted the tomcat plugin of the agent in the service tier. As of the second question, I test my project using v3.2.6 release. Do the two applications need to open JMX or configure application.yml of collector?


feat: typescript refactor

Change summary is under the line below, but the key change is obviously porting the codebase almost entirely to Typescript. This removes the need for a seperate set of type definitions derived from JSDoc, and adds all the typical quality improvements that Typescript brings.
It is key to note that moving to Typescript changes the release process by injecting a build stage. I have tried to adapt the workflows as required, but they are untested at this point. It may be best in any case to somehow release a "canary" branch and version before committing this to main.

BREAKING CHANGE: Existing implementations that rely on former JSDoc-based types or utilise undocumented parts of the API are likely to break.
Summary:

Add an .nvmrc file
JSDoc comments and JSDoc type generation removed
./lib renamed ./src and reorganised
./src/**/*.js converted to Typescript
Most types defined in ./src/models
Default values aggregated into ./src/models
Refactor of ./src/lib/buildCSPHeaders.ts
Tests converted to Typescript and utilising ts-jest
Build process codified in package.json scripts, incorporating ./scripts/copy-release-files.js
GitHub workflows updated to use new build process
Some updates to the GitHub workflows to update action versions and use .nvmrc and native Node caching
Updated .releaserc to point semantic-release to the ./dist directory - semantic release process is mostly untested due to lack of permissions.

One potential issue with this PR is that some of my syntax (tabs vs spaces) may not align to current repo practice. I think maybe there is a follow up PR where I can implement Prettier.

Hey @sambauers thank you for undertaking this 🎉
I've been looking things over and everything looks solid! We're planning on adding ESLint & Prettier together in an upcoming PR, so I think we can accept this PR as-is and get everything back in line there.
I'm going to Merge this into a working branch to setup a dev build pipeline to test the new release configuration (thank you for updating our actions as well!)


(important for makers) Date filter doesn't work on embeds

We beta-tested the tool on our embedded site if a maker can go from "hello, I want to check who's around me, and then what kind of demand is around me!" - if they activate the requester data, and then filter by last date added "1-7 days" the list is blank.
https://opensourcemedicalsupplies.org/local-response/
Very important to actually make the embed map workable.
Thank you!

Hey @tobiasdeml, sorry for the rather late reply. I'm currently working on a fix for this! The entry age is the first filter that can be applied to both requester and maker data so a little bit more coding is required. Will keep you updated on this!

Much appreciated :)
Tobias Deml
+1<PHONE_NUMBER>
*Co-Founder/DoP&Producer      *Co-Founder/ Editor
[image: Cinema of Change]
www.prodigium-pictures.com
www.cinemaofchange.com
*Social Impact Entertainment       *
Exploring the Influence of Films
Instagram https://www.instagram.com/prodigiumpics/ |  Facebook
https://www.facebook.com/prodigium.pictures/ | Sample Work
https://www.youtube.com/playlist?list=PLqqH12KmNoZQNFN4dorjcMp6icjI8zRhf
| Brochure
https://www.dropbox.com/s/ghvrwsf5lplcyhv/PRODIGIUM PICTURES-2019.pdf?dl=0
On Wed, Jul 1, 2020 at 10:27 AM zhuochenwang<EMAIL_ADDRESS>wrote:

Hey @tobiasdeml https://github.com/tobiasdeml, sorry for the rather
late reply. I'm currently working on a fix for this! The entry age is the
first filter that can be applied to both requester and maker data so a
little bit more coding is required. Will keep you updated on this!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/findthemasks/findthemasks/issues/1079#issuecomment-652550631,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP64YDNXZ2GTFYV45NYUFZLRZNWW3ANCNFSM4OIYRHNQ
.

I think this issue is fixed through #1088. Could you check the functionality when you have the time? Thanks!

Yes, works! Thank you! :)
Tobias Deml
+1<PHONE_NUMBER>
*Co-Founder/DoP&Producer      *Co-Founder/ Editor
[image: Cinema of Change]
www.prodigium-pictures.com
www.cinemaofchange.com
*Social Impact Entertainment       *
Exploring the Influence of Films
Instagram https://www.instagram.com/prodigiumpics/ |  Facebook
https://www.facebook.com/prodigium.pictures/ | Sample Work
https://www.youtube.com/playlist?list=PLqqH12KmNoZQNFN4dorjcMp6icjI8zRhf
| Brochure
https://www.dropbox.com/s/ghvrwsf5lplcyhv/PRODIGIUM PICTURES-2019.pdf?dl=0
On Thu, Jul 2, 2020 at 11:38 AM zhuochenwang<EMAIL_ADDRESS>wrote:

I think this issue is fixed through #1088
https://github.com/findthemasks/findthemasks/pull/1088. Could you check
the functionality when you have the time? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/findthemasks/findthemasks/issues/1079#issuecomment-653162854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP64YDPV6TDUQZM27D2S3D3RZTHYRANCNFSM4OIYRHNQ
.

Glad I could help!


Tutorial basics

I have reviewed and approved the changes in the docs folder.
@veghdev if the changes in tools are ok as well, than please set the PR to approved.


fix: Web - Profile - Hovering over the zoom tool does not display a tooltip.

Details

Fixed Issues
$ https://github.com/Expensify/App/issues/44029
PROPOSAL: https://github.com/Expensify/App/issues/44029#issuecomment-2179333613
Tests

Navigate to account Settings > Profile
Click on the avatar
Click on upload photo and upload any JPEG image
Hover over the zoom tool
Verify the tooltip is displayed

[x] Verify that no errors appear in the JS console

Offline tests

Navigate to account Settings > Profile
Click on the avatar
Click on upload photo and upload any JPEG image
Hover over the zoom tool
Verify the tooltip is displayed

QA Steps

Navigate to account Settings > Profile
Click on the avatar
Click on upload photo and upload any JPEG image
Hover over the zoom tool
Verify the tooltip is displayed

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/85894871/ba6cc8f5-7afc-46e6-9781-f9d51e5da990

Android: mWeb Chrome
https://github.com/Expensify/App/assets/85894871/0088d1ca-87de-458f-a543-bb2c132b9625

iOS: Native
https://github.com/Expensify/App/assets/85894871/b1c1699c-48ca-4c6d-b67f-a557b5cbeeca

iOS: mWeb Safari
https://github.com/Expensify/App/assets/85894871/88450094-65d6-4a76-b787-7f4c716207c9

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/85894871/11feb2aa-74fd-4d54-9916-500cdb40cd89

MacOS: Desktop
https://github.com/Expensify/App/assets/85894871/c152a095-0139-4472-9503-4d094605abda

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/neil-marcellini in version: 9.0.6-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/neil-marcellini in version: 9.0.6-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Hey @Krishna2323 @neil-marcellini @ishpaul777. We have a new issue where the image zoom functionality no longer works, and my best guess for the cause is this PR.
Would you mind taking a quick look?

Thanks. I opened up the new bug for contributors as a separate issue 👍

🚀 Cherry-picked to staging by https://github.com/Julesssss in version: 9.0.6-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Cherry-picked to staging by https://github.com/Julesssss in version: 9.0.6-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.6-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.7-8 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


adds social login blog

Related Issue
Link to the Github Issue created for this blog post
Link to Google Doc
TODO
Checklist

[ ] Has cover image been added
[ ] Have all content images been added. Do they render correctly? (aspect ratio etc)
[ ] The code inside code blocks gives no errors
[ ] Check for SEO keyword?
[ ] Added call to action to link to supertokens and to link to other blogs.
[ ] Add reference to how SuperTokens solves this blog's problem (if relevant).

Remaining TODOs

[ ] ...

@DBozhinovski This has been merged, will be on the test website soon


Add Syntax highlighting section to README

Add Syntax highlighting section to README, with generic instructions, and instructions for ST3 and vim.
Addresses two points of #635.

@saveriomiroddi thanks for the great job!

@SD10 do you want to add VSCode section as well?

Yes, I will do VSCode but it shouldn’t block merging this
Sent with GitHawk


@ maximn's repos

@maximn, my sbt launcher fails to load coding-interview-questions-scala and TaxCalculator with errors like this:
error: bad symbolic reference. A signature in Implicits.class refers to term Remove
in package sbt which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling Implicits.class.

I think this is because of the old sbt versions these projects use. scala-steward would be able to check for updates if they would use a newer sbt version.

Upgraded both of them to SBT 1.2.3

:+1:


Customizable root indicators

Given a subdirectory of e.g. a git repo, the get_root function will tell you the root of that repo.  Pretty straightforward goal, and there are some tests already.
For this issue,

Modify get_root to take an iterable (typing.Iterable[str]) of indicators.  Using this type, list, set, and tuple should all work.
Default to the current ROOT_INDICATORS
Add a test that shows the customization works
Submit a PR!

Quick development instructions

git clone clone https://github.com/python-packaging/tuber
cd tuber
make venv
source .venv/bin/activate
make test (runs unittest)

Make sure the code is formatted (make format) and types are good (make lint) before submitting a PR to save iteration time waiting on CI.

Hi @thatch, could you assign the issue to me?

Let me know if there's anything I can help with on this one as well.


Dedupe root domains and www subdomains

I'd argue that www is more of an implementation detail, and that they could all just be listed as the root domain, with services that use the list knowing to look at www or the root.
But in either case, listing dod.mil and www.dod.mil on two separate rows probably isn't correct, as they are the same site, so the list should just have one of those.

Thanks for suggesting this! I agree that the www and the root domain SHOULD be the same thing, and in many cases they are, so I will try to do some de-duping.
Unfortunately, I've seen a few too many cases of weirdly configured .mil domains, so I don't think I can get away with just adding the root domain when the scraper finds a www. For example, nslookup fails on dfas.mil, but works on www.dfas.mil. The same happens for a lot of unit sites, like wood.army.mil. I don't even know what to do with  a domains like mail.mil, which is actively used for email, but only respond to https (not http) on the subdomain web.mail.mil. I suppose I should also be de-duping domains like www1.nga.mil which responds with the same content as www.nga.mil, (with nslookup on nga.mil failing) but there's no reason assume this will always be so.
Anyway, I've clearly got my work cut out for me. How do you handle checking for duplicates on the .gov side?

After further thought, you've changed my mind on this.
Every browser that I checked is smart enough to try the www when the root domain fails. Also, checking every domain to look for cases where the www and the root domain serve different content would be the job for a different tool.
I think the code changes I committed last night solve the issue for the vast majority of cases, so I'm going to go ahead and close this. There are still a few duplicates, like that www1 example, but I may just have to handle those manually.
Thanks for taking a look at this!

One thing you might look at is the revamped site-inspector, which I helped with and which looks at all 4 "endpoints" for a domain -- https://www, http://www, https://, and http:// -- and tries to figure out which is "canonical" and what the overall behavior is.
I know I'll at least be running the .mil domain list through it before long!

And relatedly, something I've put some work time into is https://github.com/18F/domain-scan, which uses site-inspector and a couple other tools to output data and reports.


Cleaned up getting-started docs, added more docs for Querier.

Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

@s-urbaniak Adding this section specially for you: https://deploy-preview-1541--thanos-io.netlify.com/components/query.md/#querier-use-cases-why-do-i-need-this-component
Still in progress, but early feedback would be nice (:


Mutations are bad mmmkay (refactor to immuatable)

Now that I'm finally integrating this with my react project, I've quickly realized that lot of the ways we're doing things is not very compatible with react.  Had a very helpful conversation with a generous individual on SE
Basically, the conclusion is that some significant refactors are needed:
1: make all properties readonly (field interfaces & concrete objects?)
2: change all DML operations to return a new shallow copy, instead on mutating the instance
so instead of acc.update(true) mutating the properties directly, it would return a shallow copy with any changes spread into it.

made the generated Fields interfaces readonly.  Added a method toImmutable to deep clone and cast object back to these interfaces.
This allows you to have the best of both worlds.  My approach for react is to store a mutable instance as a class property, and then clone a immutable for the state.    This way you can use the mutable instance to update the immutable, with 0 risk of accidentally modifying state.

This could still be greatly improved but is ok for now... I think the 2.0 would completely get rid of the
RestObject base classes and just work off a more functional approach by defining mapping tables.  This would also lay the foundation for Typed SOQL queries


The Mountain or lake is uniquely identifiable. Duplicate names are extended with information to clarify which lake/mountain is meant (e.g. Pfäffikon SZ)

Seems to be really difficult. Let's do this in M4/Sprint 2


Add prow test jobs for release

We have prow jobs with matrix of Kubernetes versions for pull requests. We need to add the same matrix for release jobs as well.
The release job needs to be triggered when we're cutting a release (this can be detected by changes to charts dir in the PR)
ref: https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes-sigs/secrets-store-csi-driver/secrets-store-csi-driver-config.yaml#L597

/assign

/remove-lifecycle stale


PuppetPadlocal  ts项目启动报错timestamp ??= 0;

今天搭建一个ts项目时，初始运行就报了这个错误

运行环境
node :v14.5.0
wechaty: "^1.10.4",
wechaty-puppet-padlocal: "^0.4.2"
typescript:6.14.5
ts.config.json 为init默认设置
是否要配置什么

Have you ever made padlocal worked before?
If not, you'll have better chance get it working if starting from here --
https://github.com/padlocal/wechaty-puppet-padlocal-demo/issues/13
Took me quite an effort to get the versions that works. YMMV.

it is okay,thank you

应该吧 node 16+ 可以


Add support for knowing what pipelinerun/taskrun is running the task steps

Expected Behavior
I can tell within my task environment which pipelinerun and/or taskrun it's running for
Actual Behavior
Currently I don't see a way for my task steps to know what the pipelinerun or taskrun is associated with it.  My use case is that I emit some output to a chatbot but would like to reference the run as part of the message to make it easier for other people to check the logs of the pipelinerun.
How this is exposed to the container could be special new variables (i.e. $(tekton.pipelinerun-id)),  a workspace w/ a directory structure (i.e. /proc/ lite), environment variables, or something like the ec2 metadata api (seems a bit over the top but don't know?).
Additional Info

Kubernetes version: v1.15
Tekton Pipeline version: v0.12.0

Thanks for the issue!  I'm going to close this a duplicate of https://github.com/tektoncd/pipeline/issues/1522 since I think it's discussing the same problem.  Feel free to reopen if you think this issue is different enough to warrant its own problem statement though.


implement fit function

Hi Saurav,
when I tested the function for larger values of k for iris dataset, it is throwing an error.
from sklearn import datasets iris = datasets.load_iris() X = iris["data"] fit(X, 20)
The error is
`---------------------------------------------------------------------------
UnboundLocalError                         Traceback (most recent call last)
 in
2 iris = datasets.load_iris()
3 X = iris["data"]
----> 4 fit(X, 20)
~/Documents/MDS_UBC/block_5/labs/COLAB_SW/Kmeans_python/Kmeans_python/fit.py in fit(X_train, k, n_init, max_iter)
167         i = i + 1
168
--> 169     return centers_final, labels_final
UnboundLocalError: local variable 'centers_final' referenced before assignment`
It looks like for higher values of k, the compute_distance(X, centers) returns nans. So the below line is always false and breaks the code.
if  updated_inertia < inertia:
Could you please have a look?
Thanks

Here is how I managed to find a solution. Please have a look and confirm if this can be done.

The random initialization is initializing centers that are not in the samples. Hence, for larger k, there are some clusters which has zero samples. This results in nan
I suggest replacing the randonm initialization function with the following code
centers = X[np.random.choice(range(n_samples), k, replace=False), :]
This ensures that the random centers are chosen from the dataset itself.
This now works for any k

I think this makes sense, since if a cluster centre is initialized far from the data, it will never be assigned points to begin with, and so will never move in future iterations


verilator: test that names are mangled so as to avoid  downstream con…

Updated test so that it causes a naming conflict with Verilator.
Chisel could mangle names to avoid naming conflicts downstream. Perhaps by always adding a chisel_ prefix to all symbols, so that "iterator" becomes "chisel_iterator"?
I think it would be annoying to have to enumerate ALL potential keyword conflicts.

This is the error from the test after the modification:
verilator --cc doohickey.v --assert -Wno-fatal -Wno-WIDTH -Wno-STMTDLY --trace -O1 --top-module doohickey +define+TOP_TYPE=Vdoohickey +define+PRINTF_COND=!doohickey.reset +define+STOP_COND=!doohickey.reset -CFLAGS -Wno-undefined-bool-conversion -O1 -DTOP_TYPE=Vdoohickey -DVL_USER_FINISH -include Vdoohickey.h -Mdir /home/oyvind/src/chisel-testers/test_run_dir --exe /home/oyvind/src/chisel-testers/test_run_dir/doohickey-harness.cpp
%Error: /home/oyvind/src/chisel-testers/test_run_dir/doohickey.v:14: Symbol matching C++ common word reserved word reached emitter, should have hit SYMRSVDWORD: 'iterator'
%Error: Exiting due to 1 error(s)
%Error: Command Failed /usr/local/bin/verilator_bin --cc doohickey.v --assert -Wno-fatal -Wno-WIDTH -Wno-STMTDLY --trace -O1 --top-module doohickey '+define+TOP_TYPE=Vdoohickey' '+define+PRINTF_COND=!doohickey.reset' '+define+STOP_COND=!doohickey.reset' -CFLAGS '-Wno-undefined-bool-conversion -O1 -DTOP_TYPE=Vdoohickey -DVL_USER_FINISH -include Vdoohickey.h' -Mdir /home/oyvind/src/chisel-testers/test_run_dir --exe /home/oyvind/src/chisel-testers/test_run_dir/doohickey-harness.cpp
- should be able to compile the cpp code *** FAILED ***
  java.lang.AssertionError: assertion failed:
  at scala.Predef$.assert(Predef.scala:170)
  at chisel3.core.assert$.apply(Assert.scala:73)
  at chisel3.iotesters.chiselMain$.compile(ChiselMain.scala:75)
  at chisel3.iotesters.chiselMain$.chisel3$iotesters$chiselMain$$elaborate(ChiselMain.scala:129)
  at chisel3.iotesters.chiselMain$$anonfun$1.apply(ChiselMain.scala:169)
  at chisel3.iotesters.chiselMain$$anonfun$1.apply(ChiselMain.scala:169)
  at scala.util.DynamicVariable.withValue(DynamicVariable.scala:58)
  at chisel3.iotesters.chiselMain$.apply(ChiselMain.scala:168)

Is iterator a Verilog keyword?

Not that I know, but it conflicts with C++:
http://www.cplusplus.com/reference/iterator/iterator/

feb. 2017 21:41 skrev "Donggyu"<EMAIL_ADDRESS>
Is iterator a Verilog keyword?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ucb-bar/chisel-testers/pull/116#issuecomment-282846365,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACq05lJIDAKCQY2b_dP7SGhlNcbzIR_Rks5rgzTugaJpZM4MM1Je
.

I think it's a verilator issue. It needs to handle all name conflicts from verilog to c++.

Makes sense to me.

feb. 2017 21:52 skrev "Donggyu"<EMAIL_ADDRESS>
I think it's a verilator issue. It needs to handle all name conflicts from
verilog to c++.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ucb-bar/chisel-testers/pull/116#issuecomment-282849240,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACq05rCQ5pSrL01_ojddIT5-tHZxtH-lks5rgzd8gaJpZM4MM1Je
.

Probably a Verilator issue.


ClientConfig.RawConfig(..) does not return the merged config as documented

I have just stumbled across this same issue #220, #735 with clientcmd.NewNonInteractiveDeferredLoadingClientConfig which uses DirectClientConfig under the hood.
When retrieving the RawConfig the config is first merged e.g. merged_client_builder.go#L86-L93
However, it then returns the DirectClientConfig.RawConfig, which results in the Config without overrides.
The Interface ClientConfig implemented by DirectClientConfig explicitly mentions that RawConfig(..) should return the merged config.
Since the original issue is quite old could you kindly confirm if this is a documentation issue or indeed a bug in the implementation?

closing, as this repository is merely a mirror.


Link to per target coverage report

We're looking at adding additional split-by-target coverage reports as part of https://github.com/google/oss-fuzz/issues/7015.
We should link to that instead.

@Navidem are there any blockers here to doing this now?

I have a thought here -- I think we should add a feature for selecting which coloring/links one would like. I find both project-wide and fuzzer-specific to be useful. We have pushed some interaction capabilities recently and having the ability to select coloring + links would fit well with that. @AdamKorcz do you have some thoughts here?

For the purposes of the v1 milestone, perhaps we can start simple here? Or is it not much more work at all to support this multipel coloring/links?

#160 uses the per-target links for fuzzer details section.
It makes sense to link to per-target coverage when providing details on a specific fuzz target. For the overall project overview, total coverage link fits better.

For the purposes of the v1 milestone, perhaps we can start simple here? Or is it not much more work at all to support this multipel coloring/links?

let's start simple then we can configure it properly across the project such that the user can manage which coverage profile should be used in various places.

closing this as #164 adds support for per-target coverage report to each fuzzer's report.


Button and/or Image

Hi everyone,
Is there any way we can add a combination text, image and button in the editor. Texts are just text but for button and images, when users click we do need to do something.
I get we can add a text with slyle that appears as a hyper link and handle mouse down events for that. But wondering about image or button etc.
Regards,
Arash

No.
A longshot option you might try is to add a child Button control to Scintilla (or just place one over top of it) and watch an event like UpdateUI to constantly recalculate the position of the button based on the current scroll values, etc. But like I said, that would be trying to use Scintilla in a way it was never intended so it would be a longshot.


Adding invalid/duplicate contributor to a collection does not show an error. (bug 600205)

https://bugzilla.mozilla.org/show_bug.cgi?id=600205
Adding an invalid/duplicate contributor would not show an appropriate error.
This has been fixed by:

Making sure that users.ajax returns success with data or failure with an appropriate message.
collections.js handles the response accordingly and displays the returned message in case of failure.

Have made the required changes. Added tests as well.
For some reason Github won't add my latest commit to this pull request so I am providing you with the link to the commit:
https://github.com/vaidikkp/zamboni/tree/299fa70d2251d28d89d9e2e1c8f197c87682e7cd

Changes made.
Please review: https://github.com/vaidikkp/zamboni/commit/86ea1d1712d6bd4dcdb0af376d2d2e531071c8c3

4348c22


chore(release): v0.1.1

Version bump in package.json for release v0.1.1

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Supported PHP Versions

I came to report an issue while compiling the extension using php 7.3.16.

The ZEND_THIS constant
Seems like the constant ZEND_THIS is not available there. Before this value used to be fetched with getThis() in this repo. With php 7.4 it works. Replacing ZEND_THIS with getThis() solves the issue on php 7.3.

The zend_std_write_property() call
Another thing quite common in this repo is the following line:

value = zend_std_write_property(object, member, value, cache_slot);

This happens, for example, here: https://github.com/joseph-montanez/raylib-php/blob/89a9f1fe951897ae31a9df4bc8af7ce9db20d5b1/raylib-charinfo.c#L231
This seems to be problematic on php 7.3.16 as zend_std_write_property() returns void and value is a zval, passed for writing as the third parameter. Removing the assignment seems to solve the issue.
Note: I'm running MacOS Catalina. But this seems to be independent from build environment.

What is the minimum version supported?
I'd be happy to send out some pull requests if php 7.3 is supposed to be supported. I'm not a C expert but I think I can help a bit there.
Cheers!

Thanks, initially I was limiting the scope to just PHP 7.4, as the stubs require PHP 7.4 due to type signatures in method calls. However, supporting PHP 7.3 via the C code isn't an issue. Thanks for catching this I'll make an update to change this and test on 7.3.

Just an update I've been trying on and off to support 7.3. While I am able to build, the property writers are segment faulting. So this will have to be an ongoing effort. I don't want this to slow down the binding's implementation, so, for now, I'll push 7.3 support to a different branch. I looked at how php-src handles 7.4 and 7.3 and they don't, they just keep completely different repos, so if-defs are not littered all over the place for various php versions.


New-SSHSession : Key exchange negotiation failed. SecurityError: (Renci.SshNet.SshClient:SshClient)

Windows Version
Windows 10 Version 1803 (OS Build 17134.112)
Powershell Version
Major  Minor  Build  Revision
5             1      17134  112
The problem is when I try to create an ssh session, I get the below.
$cred = New-Object System.Management.Automation.PSCredential ("user", (ConvertTo-SecureString "pass" -AsPlainText -Force))
$guestvm    = "<IP_ADDRESS>"
New-SSHSession -AcceptKey -ComputerName $guestvm -Credential $cred

New-SSHSession : Key exchange negotiation failed.
At line:1 char:1
+ New-SSHSession -AcceptKey -ComputerName $guestvm -Credential $cred
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : SecurityError: (Renci.SshNet.SshClient:SshClient
   ) [New-SSHSession], SshConnectionException
    + FullyQualifiedErrorId : SSH.NewSshSession

Ensure the target server is not using elliptic curve keys. Some Linux distros have moved to them but SSH.Net library does not support them
Sent from my iPhone

On Jun 15, 2018, at 3:45 PM, aristosv<EMAIL_ADDRESS>wrote:
Windows Version
Windows 10 Version 1803 (OS Build 17134.112)
Powershell Version
Major Minor Build Revision
5 1 17134 112
The problem is when I try to create an ssh session, I get the below.
$cred = New-Object System.Management.Automation.PSCredential ("user", (ConvertTo-SecureString "pass" -AsPlainText -Force))
$guestvm    = "<IP_ADDRESS>"
New-SSHSession -AcceptKey -ComputerName $guestvm -Credential $cred
New-SSHSession : Key exchange negotiation failed.
At line:1 char:1

New-SSHSession -AcceptKey -ComputerName $guestvm -Credential $cred

  + CategoryInfo          : SecurityError: (Renci.SshNet.SshClient:SshClient
 ) [New-SSHSession], SshConnectionException
  + FullyQualifiedErrorId : SSH.NewSshSession

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Try to remove:
Get-SSHTrustedHost | Remove-SSHTrustedHost

Thank you.  Get-SSHTrustedHost | Remove-SSHTrustedHost worked for me.  Where is the list of trusted hosts maintained?  And is there a way that New-SFTPSession -AcceptKey can automatially update this trusted key?

It is kept in the registry, in the PSv7 I’m thinking of using a JSON file to keep the host fingerprint pairs.

It fixed my problem too while connecting to a brocade sw in SSH.
Thanx!

Adding -force at the end of the New-SSHSession line fixed it for me.
New-SSHSession -ComputerName $HostnameorIP -Credential $LoginCreds -AcceptKey:$true -force

Adding -force at the end of the New-SSHSession line fixed it for me.
New-SSHSession -ComputerName $HostnameorIP -Credential $LoginCreds -AcceptKey:$true -force

That means the host key changed
Sent from my iPhone

On Jan 6, 2021, at 9:05 AM, DjBangAdmin<EMAIL_ADDRESS>wrote:

Adding -force at the end of the New-SSHSession line fixed it for me.
New-SSHSession -ComputerName $HostnameorIP -Credential $LoginCreds -AcceptKey:$true -force
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

That means the host key changed
Sent from my iPhone

On Jan 6, 2021, at 9:05 AM, DjBangAdmin<EMAIL_ADDRESS>wrote:

Adding -force at the end of the New-SSHSession line fixed it for me.
New-SSHSession -ComputerName $HostnameorIP -Credential $LoginCreds -AcceptKey:$true -force
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.


Merging Personal Fork into API-Content Branch

Initial merge of my personal fork into api-content branch.
Instructions:

npm install packages
check the api page

Something went wrong with PR preview build please check


Italic not working in top of Terminology page

What happened?
Looks like the intention is for the text to be all italic, but where the source breaks it is not working.

Thanks for noticing.


Allow customizing content-type when file istext/html

This line forces the response content type to be text/plain when the object is of type text/html. It would be nice to have a customizable option. My use case with storage is storing some user-generated html files, and this blocks me from getting the actual rendered page but the html source code.

Hey @qiushiyan are you self hosting storage-api and want this to be configurable?

Please can you explain why you are rewriting the content-type text/plain when the file is text/html. This is super annoying as it means I cant host html files eg for openapi docs using the storage api. Is there anyway to override this behaviour?

I actually found a workaround for this but it is still annoying.


Add support for [type], [start] and [reversed] attributes for the ol tag

Hi,
we use the ordered bullet lists on our site, I know we can use the CSS "list-style-type" property to do the equivalent but for accessibility needs we have to use the [type], [start] and [reversed] attributes.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/ol
Thank you

Thx for the report. Seems to be reasonable, I'm going to have look into that during the next few days...

@YanickRoss do the changes as provided in PR #58 meet your requirements? Thx for your feedback!

@ohader yes that's exactly what we want. thank you


Add blake2s hint

Closes #378

@lana-shanghai @tcoratger Hi, could you assist me with the failing CI? I run make test locally and it already pass all the unit tests.


[BUG] Spark job failure cannot be caught in Flint

What is the bug?
When a Spark job failed, the PPL query will be hanging forever.
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to Workbench page. For example https://search-flint05-sanity-h5if7yelmxws5hc35cap2lauf4.us-east-1.es-integ.amazonaws.com/_dashboards/app/opensearch-query-workbench#/
Execute a PPL query which could cause job fail: such as following PPL query.

source = myglue_test.default.http_logs | stats avg(size) by clientip

The query group by a high cardinality column will trigger a writing failure.

The query will be running forever in workbench

What is the expected behavior?
Failed with error message
What is your host/environment?

OS: [e.g. iOS]
Version [e.g. 22]
Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

@ykmr1224 @noCharger , please help to transit to proper project if it doesn't belong to this project.


[Bug] Providing tool response back to llm for output generation is broken for llama3.1 8B

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[x] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.

Describe the bug
I am using lang graph for tool calling with llama-3.1 8B Instruct model.
The model responses work well until tool calling section. It gives an internal server error when the tool response is sent back to llm for producing an output.
Reproduction
Steps to reproduce:
Attaching the code snippet as an example
from langchain_openai import ChatOpenAI
model = ChatOpenAI(
    model = model_name,
    temperature = 0,
    base_url=model_url,
    api_key="EMPTY",
    disable_streaming=True,
)

# Define tools
from langchain_core.tools import tool
from typing import Literal
@tool
def get_weather(city: Literal["nyc", "sf"]):
    """Use this to get weather information."""
    if city == "nyc":
        return "It might be cloudy in nyc"
    elif city == "sf":
        return "It's always sunny in sf"
    else:
        raise AssertionError("Unknown city")

tools = [get_weather]

# Define the graph
from langgraph.prebuilt import create_react_agent
graph = create_react_agent(model, tools=tools)

# Printing
def print_stream(stream):
    for s in stream:
        message = s["messages"][-1]
        if isinstance(message, tuple):
            print(message)
        else:
            message.pretty_print()

# Run:
inputs = {"messages": [("user", "what is the weather in sf")]}
print_stream(graph.stream(inputs, stream_mode="values"))

# OUTPUT:
================================ Human Message =================================

what is the weather in sf
================================== Ai Message ==================================

<function=get_weather>{"city": "sf"}</function>
Tool Calls:
  get_weather (0)
 Call ID: 0
  Args:
    city: sf
================================= Tool Message =================================
Name: get_weather

It's always sunny in sf
------ Server Error ---

Environment
Docker image: openmmlab/lmdeploy:v0.6.1-cu11

Command: lmdeploy serve api_server --server-port 8000  meta-llama/Llama-3.1-8B-Instruct

Error traceback
September 30 19:41:33.390
Traceback (most recent call last):
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/uvicorn/protocols/http/h11_impl.py", line 406, in run_asgi
September 30 19:41:33.390
    result = await app(  # type: ignore[func-returns-value]
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 60, in __call__
September 30 19:41:33.390
    return await self.app(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/fastapi/applications.py", line 1054, in __call__
September 30 19:41:33.390
    await super().__call__(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/applications.py", line 113, in __call__
September 30 19:41:33.390
    await self.middleware_stack(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/middleware/errors.py", line 187, in __call__
September 30 19:41:33.390
    raise exc
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/middleware/errors.py", line 165, in __call__
September 30 19:41:33.390
    await self.app(scope, receive, _send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/middleware/cors.py", line 85, in __call__
September 30 19:41:33.390
    await self.app(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 62, in __call__
September 30 19:41:33.390
    await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/_exception_handler.py", line 62, in wrapped_app
September 30 19:41:33.390
    raise exc
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/_exception_handler.py", line 51, in wrapped_app
September 30 19:41:33.390
    await app(scope, receive, sender)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/routing.py", line 715, in __call__
September 30 19:41:33.390
    await self.middleware_stack(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/routing.py", line 735, in app
September 30 19:41:33.390
    await route.handle(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/routing.py", line 288, in handle
September 30 19:41:33.390
    await self.app(scope, receive, send)
September 30 19:41:33.390
  File "/opt/py3/lib/python3.10/site-packages/starlette/routing.py", line 76, in app

Hi, @S1LV3RJ1NX may check if https://github.com/InternLM/lmdeploy/pull/2558 meets your need.

Similar issue #2366


Crash upon joining a private match or creating a match

Bug
When I try to open a private match or join a match from the server browser it crashes on the latest northstar version
Campaign maps on northstar work fine as well as vanilla Titanfall 2 both Campaign/Multiplayer
Steps to reproduce

Start northstar
Try to join a match from the server browser or create one

Specifications

Northstar version: 1.29.0
Platform: Origin (via Proton Experimental under Linux)

nslog2024-11-30 15-17-52.txt

Crash in material systems without additional mod... Weird.
...
[2024-11-30] [15:20:02] [NORTHSTAR] [error] -------------------------------------------
[2024-11-30] [15:20:02] [NORTHSTAR] [error] Northstar has crashed!
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	Version: <IP_ADDRESS>
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	EXCEPTION_ACCESS_VIOLATION
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	Attempted to write to: 0x2fb0
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	At: materialsystem_dx11.dll + 0x284f9
[2024-11-30] [15:20:02] [NORTHSTAR] [error] Callstack:
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	materialsystem_dx11.dll + 0x284f9
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	materialsystem_dx11.dll + 0x28769
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	materialsystem_dx11.dll + 0x7700d
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	materialsystem_dx11.dll + 0x75fd5
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	materialsystem_dx11.dll + 0x8808f
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	tier0.dll + 0x1ba3b
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	kernel32.dll + 0x1482d
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	ntdll.dll + 0xf44f
[2024-11-30] [15:20:02] [NORTHSTAR] [error] 	NorthstarLauncher.exe + 0xfffffffec0000000
...

If you go back to v1.28.6, does it work again?

If you go back to v1.28.6, does it work again?
Nope, still largely the same

Log:
nslog2024-12-01 17-31-45.txt

What version were you on when it still worked?

What version were you on when it still worked?

I don't really know, haven't re-installed northstar since long ago but I'll try all the builds I can try and report back

Assuming you still have the old logs, you can just check the previous ones for the version. The first line printed in the long contains the version number of Northstar.


TokenReplace on Web.Config

Hi,
Maybe I've misunderstood how this is supposed to work so let me explain....
I am using the TokenReplace task to replace some tokens in my web.config file before deployment (TFS 2015 Release Defintion).
The TokenReplace task produces the error:
[error]Could not find files with glob [\\sevstfs\_work\2\a\_PublishedWebsites\Starlight\*.config].
However, I have confirmed that the file is and was, there.
My Environment variables in the definition are:
And my web.config tokens are here:

Not sure if it is because I cannot use the web.config or whether it's something else.
Any help appreciated.
thanks,
KS

Hi @perp1exed - I've not tried replacing tokens on a UNC path. I'd suggest that you copy the file to the build agent (use File copy task), do the replacement and then copy it to the target machine.

The UNC path is pointing to the folder on the build agent (which is the tfs
server).
I am using the TokenReplace task in the Release definition. Is it safe to
say that it is running on the build agent? In which case specifying the
local directory path should work?
Cheers,
Ks
On Friday, 9 September 2016, Colin Dembovsky<EMAIL_ADDRESS>wrote:

Hi @perp1exed https://github.com/perp1exed - I've not tried replacing
tokens on a UNC path. I'd suggest that you copy the file to the build agent
(use File copy task), do the replacement and then copy it to the target
machine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/colindembovsky/cols-agent-tasks/issues/28#issuecomment-246035612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVFNa5ihrB3CdF0bNpagLihuoOthn2Teks5qocZBgaJpZM4J5XCn
.

--
regards,
Khurram

The build runs on the machine that the build agent is installed on - so if you can access that folder via a regular path (like c:\builddrops...\web.config) then that should work!

So the pre-defined variables do not work right?
For example, $(Build.StagingDirectory)\drop does not work.
I am trying to avoid having to explicitly set the source folder.
Thanks,
KS

I found a similar issue. It seems that if you point the task directly at a file it will not work but if you point the task at the folder which contains the file and then specify "*.config" (or whatever the file type is) the task will complete.

Hi @perp1exed - I'm still not following your problem clearly. Can you send a screenshot of your task config so that I can see how you're configuring it?