Warn Windows 7 users about old TLS

An eyepatch for https://github.com/rust-lang/cargo/issues/5066.
@retep998 what would the best way to check for windows 7 specifically? Currently I use just #[cfg(windows)], which might give false positives on 8 and 10.

This seems like it'd be helpful: https://msdn.microsoft.com/en-us/library/windows/desktop/dn424961(v=vs.85).aspx.

According to,
https://support.microsoft.com/en-us/help/3140245/update-to-enable-tls-1-1-and-tls-1-2-as-a-default-secure-protocols-in
Looks like the patch for windows requires or set a DefaultSecureProtocols registry entry. Maybe we can check for that?
And thanks for making this patch!

Amusingly, trying to test out this fix on win7 with
cargo install --git https://github.com/matklad/cargo --branch win7-y-u-no-tls

fails with this very error =P

Ok, I've checked that the current patch works as expected on windows 7.
I've looked into detecting registry key/windows version a bit, but I've failed to find a smiple API in winapi-rs to do this, and I don't feel confident about using complex windows API, so it would be great if someone with more windows experience implements a version check =)

The KB article declares that the issue is for Windows Server 2012 which is the server equivalent of Windows 8, but doesn't list Windows 8 itself as affected. But this is probably because Windows 8 became unsupported some time before that update was released, so it probably is affected and including it in the check should not be a problem.

The reg key doesn't exist on Windows 10 and Windows 10 works without it, so just testing the reg key's existence independent of the OS version will not be correct.

So  IsWindows8Point10OrGreater returning FALSE is probably all you need.

Unfortunately IsWindows8Point10OrGreater  is not yet in winapi-rs. :-(
I tried to use GetVersionEx directly:
    use winapi::um::winnt::OSVERSIONINFOW;
    use winapi::um::sysinfoapi::GetVersionExW;
    let mut os = OSVERSIONINFOW {
        dwOSVersionInfoSize: std::mem::size_of::<OSVERSIONINFOW>() as u32,
        dwMajorVersion: 0,
        dwMinorVersion: 0,
        dwBuildNumber: 0,
        dwPlatformId: 0,
        szCSDVersion: [0; 128],
    };
    unsafe { GetVersionExW(&mut os) };
    println!("{:?}.{:?}", os.dwMajorVersion, os.dwMinorVersion);

and on my win10, I get 6.2. according to the docs at https://msdn.microsoft.com/en-us/library/windows/desktop/ms724834(v=vs.85).aspx that is win8, probably something about not having a manifest file. But if we get 6.1 on win 7 we may have a workaround. (I don't have a win7 box to test on.)

@Eh2406 Yes, as I said in my comment, the function is header-only, and lies to applications without manifests.

Something like this will work, once you've figured out the manifest problem (not tested or compiled):
const WIN32_WINNT_WINBLUE_HIBYTE = 0x06;
const WIN32_WINNT_WINBLUE_LOBYTE = 0x06;

fn IsWindows8Point1OrGreater() -> bool
{
    IsWindowsVersionOrGreater(WIN32_WINNT_WINBLUE), WIN32_WINNT_WINBLUE_LOBYTE, 0)
}

fn IsWindowsVersionOrGreater(majorVersion: DWORD, minorVersion: DWORD, servicePackMajor: DWORD) -> bool {
    let conditionMask: DWORDLONG = VerSetConditionMask(VerSetConditionMask(VerSetConditionMask(0, VER_MAJORVERSION, VER_GREATER_EQUAL), VER_MINORVERSION, VER_GREATER_EQUAL), VER_SERVICEPACKMAJOR, VER_GREATER_EQUAL);

   let mut osvi: OSVERSIONINFOEXW = std::mem::zeroed();
   osvi.dwMajorVersion = majorVersion;
   osvi.dwMinorVersion = minorVersion;
   osvi.wServicePackMajor = servicePackMajor;

  VerifyVersionInfoW(&mut osvi, VER_MAJORVERSION | VER_MINORVERSION | VER_SERVICEPACKMAJOR, dwlConditionMask) != FALSE
}

@Arnavion, I can't find your comment about manifests. I am sorry if I echoed info you had already provided.

once you've figured out the manifest problem

I don't think the 'manifest problem' is trivial. I'd love to be proven wrong. I don't know of a way to add a manifest that doesn't involve a large build dependency. The win SDK for mt.exe or LLVM for its alternative. Do we want to require that for building cargo?

not tested or compiled

Yes that is a better solution, once it is converted to valid rust.

@retep998 should comment about that. It's not a problem for windows-msvc builds of course, but I don't know anything about windows-gnu (assuming the cargo.exe that ships with the windows-gnu toolchain is itself built with windows-gnu and not just the same binary as the one that ships with the windows-msvc toolchain).

If we need to deal with the manifest to lowerer the false positive rate, then perhaps we merge as is for now so it can get in 1.24.1. And add a FIXME to get dealt with wen manifests are fixed.

Agreed. The function's comment and error message should also mention Server 2008R2 and Server 2012 in addition to 7, and perhaps word it like "If you're on ... then certificate check failure might be caused..."

Having the necessary tooling from VC++/MinGW to link manifests is not an issue. The issue is having the support from Rust itself. It will involve non-trivial design work to come up with a good way to link manifests, and I doubt it will happen any time soon.
However there is a shorter term solution for manifests and that is having simple side by side manifests instead of embedded manifests. It would involve someone writing manifests such as cargo.manifest and then getting the Rust packaging stuff to shove that in the same component and folder as cargo.exe.

I think we can use rust-embed-resource or similar build script magic to include the manifest, without the design work from rustic. (If we have the tooling from VC++/MinGW installed.)
I have never had any luck getting side by side manifests to work, but it does sound like a simple solution.

Hm, it looks like we won't be able to implement the manifest solution in any reasonable time frame :-(
Could we check for the presence of HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Internet Settings\WinHttp instead, as suggested by @Eh2406?
Specifically, is this key present on Windows 8 and Windows 10?

Updated the error message to mention windows server and to be conditional on "if you are indeed on windows".
It seems unlikely to me that we will be able to implement the cargo.exe.manifest solution fast, and this I think means that this patch is the best we can hope for 1.24.1.
The failure mode here is that a user on Windows 10 might get this wrong worming if threre's genuinely something wrong with TLS. However, because we are checking for a specific code from libgit, this seams unlikely?

Thanks for the quick fix here @matklad! It sounds like this is hopefully going to be a temporary holdover until https://github.com/libgit2/libgit2/pull/4550 is sorted out, after which we'll probably just update libgit2 pronto and get that into Cargo. In that sense it seems like we'll be using this for hopefully only a few cycles, but still a few cycles in the sense that it'll get backported and such.
This looks pretty conservative but I think one final thing we may wish to do is to only warn about connections to github.com perhaps? I believe that's the main instigator for this issue, right? It may also help reduce the noise of warnings in legitimate situations as that seems to come up relatively often (certificate errors on Windows).
I'd personally also be much more comfortable backporting a warning like this as well rather than libgit2 updates, so I think for sure we should pursue this.
@matklad would you be ok switching to only warning on github.com hostnames?

@matklad would you be ok switching to only warning on github.com hostnames?

Sure will try to do this in a couple of hours!

Updated to check for GitHub specifically, and verified on the virtual machine that the message is indeed printed on an unpatched win7, and that a patched one works OK.

@bors: r+

:pushpin: Commit 81e18de has been approved by alexcrichton

:hourglass: Testing commit 81e18de756538dfb5b9029e27fe6ac7cec7db461 with merge ce87cfa6295d2562c5bb0e9404e75722e5093d38...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing ce87cfa6295d2562c5bb0e9404e75722e5093d38 to master...


Integrating with OSS-Fuzz

Greetings easywsclient developers and contributors,
We’re reaching out because your project is an important part of the open source ecosystem, and we’d like to invite you to integrate with our fuzzing service, OSS-Fuzz. OSS-Fuzz is a free fuzzing infrastructure you can use to identify security vulnerabilities and stability bugs in your project. OSS-Fuzz will:

Continuously run at scale all the fuzzers you write.
Alert you when it finds issues.
Automatically close issues after they’ve been fixed by a commit.

Many widely used open source projects like OpenSSL, FFmpeg, LibreOffice, and ImageMagick are fuzzing via OSS-Fuzz, which helps them find and remediate critical issues.
Even though typical integrations can be done in < 100 LoC, we have a reward program in place which aims to recognize folks who are not just contributing to open source, but are also working hard to make it more secure.
We want to stress that anyone who meets the eligibility criteria and integrates a project with OSS-Fuzz is eligible for a reward.
To help you getting started, we can attach our internal fuzzer for your project that you are welcome to use directly, or to use it as a starting point.
If you're not interested in integrating with OSS-Fuzz, it would be helpful for us to understand why—lack of interest, lack of time, or something else—so we can better support projects like yours in the future.
If we’ve missed your question in our FAQ, feel free to reply or reach out to us at<EMAIL_ADDRESS>Thanks!
Tommy
OSS-Fuzz Team

We can also help by starting the OSS-Fuzz integration process for you if lack of time. We just need a google email from you to send bugs to you when libfuzzer finds any.

This sounds great. What step would you like me to take to get started? Thanks.

Here is the link to how to set up a new project to OSS-Fuzz: https://github.com/google/oss-fuzz/tree/master/projects
Please don't hesitate to ask questions or concerns if you need help. Thanks. @dhbaird

I want to do this, but have been swamped in other work. It may be better if you can do the setup steps for me, that you mentioned. You can use this email address<EMAIL_ADDRESS>Thanks.

Easywsclient has been PR to OSS-Fuzz (https://github.com/google/oss-fuzz/pull/3126). Please take a look and make any changes or add fuzzers as you like.


Error when build plugin from source

Hi,
the first thanks to the nice plugin.
I have a problem. i build plugin from your source, when i run elasticsearch have an error:
{
"error": "NoClassDefFoundError[vn/hus/nlp/sd/SentenceDetectorFactory]; nested: ClassNotFoundException[vn.hus.nlp.sd.SentenceDetectorFactory]; ",
"status": 500
}
This is my source

Plz help me,
Thanks you!

You are missing vn tokenizer jars in the class path, make sure that all jar files in libs folder are in the class path.

Closed due to lack of feedback


Couchbase Enterprise Release 1.2.2

Updates package to point to csv with operator release of 1.2.2
Signed-off-by: Tommie McAfee<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file a PR against this repo and explain your need
2 For more information see here

@tahmmee, i got following error:
kubectl logs couchbase-operator-6dbc64b7cb-wlsdt -n couchbase-enterprise-vdsky scorecard-proxy
time="2020-01-14T12:11:14Z" level=info msg="Watching couchbase-enterprise-vdsky namespace."
{"level":"info","ts":1579003874.617708,"logger":"controller-runtime.metrics","msg":"metrics server is starting to listen","addr":":8080"}
{"level":"error","ts":1579003874.6189907,"logger":"controller-runtime.metrics","msg":"metrics server failed to listen. You may want to disable the metrics server or use another port if it is due to conflicts","error":"error listening on :8080: listen tcp :8080: bind: address already in use","stacktrace":"github.com/go-logr/zapr.(*zapLogger).Error\n\t/home/travis/gopath/pkg/mod/github.com/go-logr/zapr@v0.1.1/zapr.go:128\nsigs.k8s.io/controller-runtime/pkg/metrics.NewListener\n\t/home/travis/gopath/pkg/mod/sigs.k8s.io/controller-runtime@v0.4.0/pkg/metrics/listener.go:44\nsigs.k8s.io/controller-runtime/pkg/manager.New\n\t/home/travis/gopath/pkg/mod/sigs.k8s.io/controller-runtime@v0.4.0/pkg/manager/manager.go:277\nmain.main\n\toperator-sdk/images/scorecard-proxy/cmd/proxy/main.go:46\nruntime.main\n\t/home/travis/.gimme/versions/go1.13.6.linux.amd64/src/runtime/proc.go:203"}
time="2020-01-14T12:11:14Z" level=fatal msg="error listening on :8080: listen tcp :8080: bind: address already in use"

@tahmmee, it is working for me locally. Installed and scorecard passed.
PASS: Scorecard test passed
NORMAL
Test operator with scorecard                      [  OK  ]


Usage of commas as decimal points for some properties

I noticed that a comma was used for the decimal point for some properties of primitives (like position), while a dot was used for others, like for the radius of a sphere.

I noticed that a comma was used for the decimal point for some properties of primitives (like position), while a dot was used for others, like for the radius of a sphere.

Strange, I can't even use commas in there

me neither

Me neither, but the dot is replaced by a comma after submission.

Note: cComma is the standard decimal separator in France, but my operating system, keyboard, and all other settings are set to "English (US)".


trying to run "getting started"

Hello there,
I am having to run your getting started in the colab. It was configured python3 + GPU. Could you help what is going on there?

Thanks.

I have been having a similar problem does anyone have a solution for this, There are some past issues mentioning around torch and torchtext version issues.

i have been using :
!pip install torch==0.3.1
!pip install torchtext==0.2.3
that is required based on the requirement. I was able to run locally by recompiling source code. but, I had to go through many conflicts though. I am trying to run on my gpu box but, it seems that there are some conflicts on cuda version though. If we know the exact spec on configuration, I think it will be very helpful though.

Thanks I figured that out. I was trying to get the model summary, Any ideas Im using torchsummary  but no idea of the input shape


Test Scaffolding

I can't comment on the binary, but what has changed in kudu?

I can't comment on the binary, so commenting here. What has changed in kudu?

I changed it to a build that doesn't require the newest Glibc.


any plans to make it support generic http?

Hi,
seems like this connector could query an api endpoint and get response as json
it's there any plan to extend it a bit support sending headers/body and get a bit more result like status code, headers, content type?

This is not meant to be a generic HTTP connector, only to read local or possibly remote JSON files. But it's not supposed to handle any authentication for remote files.
You could try writing a custom connector for a specific HTTP API yourself. Check out this one for more examples: https://github.com/nineinchnick/trino-rest


Bug: Quote style is not applied

[X] I have verified that I am on the latest version of the Linter

Describe the Bug
When the file has an even number of straight single quotes, Linter complains that it doesn't, and won't make the straight quotes curly.
How to Reproduce
Write a file with one or more words in single quotes, and save it, triggering Linter.
Expected Behavior
Curly quotes and no complaints :-)
By the way, the "even number" test seems unwise, given that in many languages, like Dutch and English, single instances of single quotes are used, as in zo'n and doesn't.
Screenshots

Device

[X] Desktop
[ ] Mobile

Hey @RhvdG , I will need to look at the scenario in question. The Linter does look into whether or not an English contraction is being used (I cannot acvount for other languages as I don't know contractions in other languages).
Would you recommend checking the character before and after a single or double quote to see whether it should be an opening or closing smart quote? If so, how are fullwidth characters handled? If we are not dealing with fullwidth characters, how do we handle a quote character between 2 non-quote characters?
I look forward to seeing how this logic can be improved.

Hi. In Dutch, English, French, Italian (and probably in other languages as well) when a ' occurs inside a word, with [a-z,A-Z] on both sides, it's always a closing quote mark ’.
Zo’n
Don’t
l’autre
all’inizio
I am not sure what you mean by "fullwidth characters".

Full width characters are Chinese, Japanese, and Korean characters, but they are also used as English characters sometimes. Fullwidth characters have no need to have spacing added around them as they already come with it added in.

I am thinking that the following could work as an option for a replacement algorithm that is more robust:
Go through and determine opening and closing values based on what preceeds and follows the quote. When the quote is followed by a letter, it is an opening quote. If it is followed by and/or preceeded by a letter it will be a closing quote.
If the surrounding characters are not letters then it will be determined based on what the last quote was. If the last quote was an opening quote it will be a closing quote. Otherwise it will be an opening quote?

Sounds about right!

I believe I have a fix for this. It should not be too bad to implement. There are some edge cases that may need to be ironed out. But I plan to push it up and have it in the next release.

This has been merged into master now. It should take care of the problem in question and add better support. It may need modification, but it is better than it was before from my understanding.
Please let us know if there is an issue on master or in the next release.


evaluation version of gpt-3.5-turbo

Hello, I cannot correctly reproduce the test results of llava-next-video. I suspect it might be an issue with the GPT-3.5-turbo version. Different periods had different versions of GPT-3.5-turbo. Using the latest GPT-3.5-turbo-16k version significantly raises the results. The previous model did not have the GPT-3.5-turbo-16k version. So, could you please tell me which version of GPT was used for evaluation in llava-next-video?

gpt-3.5-turbo-0613

Hi~
It seems that gpt-3.5-turbo-0613 is not working. Either gpt-3.5-turbo-0613 or gpt-3.5-turbo points to gpt-3.5-turbo-0125. Because I found the evaluation scores are very high.
Any solution to use gpt-3.5-turbo-0613 API?
Thanks a lot!

hi @farewellthree , if you use gpt-3.5-turbo-0613, can you reproduce the scores in the VideoChatgpt datasets? I found it is also super higher than the score in the table. Such as the tempral score, I obtained 4.12 for LLaVA-NeXT-Video-DPO (7B). Very high!


[Update Favicon] Favicon is still the christmas tree

The christmas favicon is still active. Do we want to go back to the original favicon of the tree? Or do we want to change it to one of the new tree people with faces?

@zoobot I remember we liked this braced tree graphic. Do you want to use this as a new favicon? Or just go back to the original favicon? This seems to be more on brand of maintenance of trees.


[AM-84] New logo

New logos

Amplify
SLNSW
NSW Government

Inserted into header templates & content
Adjusted styles

@sameera207 will do


Media Player recording data usage

I've noticed that the Android Media player is adding to the Data Usage used counter for our application when setting the player to use getProxyUrl()
I believe this is because the Android Media Player thinks it's playing from an internet source, even though it is streaming locally from the device. This is also happening within your sample application. Is there any way to stop this from happening as it looks like the application is using much data that it really is. This is also apparent in the Android Studio Network tool which is showing a network usage EVENT though I have no data connection and if I have airplane mode off.
I am unsure whether this is more related to this library or it's completely down to Android. I've been testing this on Android 6 Nexus 5

Is the correct implementation, that if a user is connected to the internet then the file will be downloaded regardless as to whether it is in the cache or not? That is what I am currently seeing on my Nexus 6P, even when I use your sample application and Android Studios Network Statistics panel.
Unless the Network panel can't differentiate between external and local network communication. Which could very possibly be the case.

After some investigation adding connection.setRequestMethod("HEAD"); into HttpUrlSource.openConnection may help, but it doesn't fix the issue.
I am more convinced Android is reporting the data incorrectly.

@brittonjg You can check was the file fully cached by method HttpProxyCacheServer.isCached(String url). If it was cached you can use local file uri directly to file instead of proxied url.


Amplify add new env failure with error : HostedUIProvidersCustomResourceInputs Custom::LambdaCallout

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[x] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
yarn
If applicable, what version of Node.js are you using?
v15.12.0
Amplify CLI Version
4.45.2
What operating system are you using?
Mac
Amplify Categories
auth, storage, api, hosting
Amplify Commands
env, publish
Describe the bug
I am trying to add new env for the amplify backend and am getting the below error. :
CREATE_FAILED      HostedUIProvidersCustomResourceInputs Custom::LambdaCallout Mon Mar 22 2021 19:33:12 GMT-0400 (Eastern Daylight Time) Failed to create resource. See the details in CloudWatch Log Stream: 2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4
CREATE_IN_PROGRESS HostedUIProvidersCustomResourceInputs Custom::LambdaCallout Mon Mar 22 2021 19:33:12 GMT-0400 (Eastern Daylight Time) Resource creation Initiated
CREATE_IN_PROGRESS HostedUICustomResourceInputs          Custom::LambdaCallout Mon Mar 22 2021 19:33:12 GMT-0400 (Eastern Daylight Time) Resource creation Initiated
CREATE_COMPLETE    UserPoolClientInputs                  Custom::LambdaCallout Mon Mar 22 2021 19:33:11 GMT-0400 (Eastern Daylight Time)
CREATE_IN_PROGRESS UserPoolClientInputs                  Custom::LambdaCallout Mon Mar 22 2021 19:33:10 GMT-0400 (Eastern Daylight Time) Resource creation Initiated
⠇ Updating resources in the cloud. This may take a few minutes...

CREATE_FAILED amplify-someapplication-envtwo-192949-authsomeapplication10ec9a3f-N3HNQXJZ0DNP AWS::CloudFormation::Stack Mon Mar 22 2021 19:33:14 GMT-0400 (Eastern Daylight Time) The following resource(s) failed to create: [HostedUIProvidersCustomResourceInputs].
⠦ Updating resources in the cloud. This may take a few minutes...

UPDATE_ROLLBACK_IN_PROGRESS amplify-someapplication-envtwo-192949 AWS::CloudFormation::Stack Mon Mar 22 2021 19:33:29 GMT-0400 (Eastern Daylight Time) The following resource(s) failed to create: [hostingS3AndCloudFront, authsomeapplication10ec9a3f].
CREATE_FAILED               hostingS3AndCloudFront               AWS::CloudFormation::Stack Mon Mar 22 2021 19:33:29 GMT-0400 (Eastern Daylight Time) Resource creation cancelled
CREATE_FAILED               authsomeapplication10ec9a3f               AWS::CloudFormation::Stack Mon Mar 22 2021 19:33:28 GMT-0400 (Eastern Daylight Time) Embedded stack arn:aws:cloudformation:us-east-1:239888062377:stack/amplify-someapplication-envtwo-192949-authsomeapplication10ec9a3f-N3HNQXJZ0DNP/bcf0b9f0-8b66-11eb-b644-0a8082d84415 was not successfully created: The following resource(s) failed to create: [HostedUIProvidersCustomResourceInputs].

From Log stream :
2021-03-22T23:33:10.456Z	f141c0f0-b781-433a-bdcd-aa5122b4cb5d	INFO	Response body:

{
    "Status": "FAILED",
    "Reason": "See the details in CloudWatch Log Stream: 2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4",
    "PhysicalResourceId": "2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4",
    "StackId": "arn:aws:cloudformation:us-east-1:[ACCOUNT_NUMBER]:stack/amplify-someapplication-envtwo-192949-authsomeapplication10ec9a3f-N3HNQXJZ0DNP/bcf0b9f0-8b66-11eb-b644-0a8082d84415",
    "RequestId": "5b4f5ad6-5342-4bd7-8984-638cdf17bdf1",
    "LogicalResourceId": "HostedUIProvidersCustomResourceInputs",
    "NoEcho": false,
    "Data": {
        "err": {
            "message": "client_id, client_secret and authorize_scopes are all required idp details.",
            "code": "InvalidParameterException",
            "time": "2021-03-22T23:33:10.414Z",
            "requestId": "b5cef6fd-40d0-474a-9005-84cacf7c8f51",
            "statusCode": 400,
            "retryable": false,
            "retryDelay": 87.55256440291616
        }
    }
}

Expected behavior
New amplify env should be added without issues
Reproduction steps

Amplify init
Add amplify env with below services

Current Environment: envone

| Category | Resource name      | Operation | Provider plugin   |
| -------- | ------------------ | --------- | ----------------- |
| Auth     | somename00000000 | Create    | awscloudformation |
| Storage  | somenamestorage  | Create    | awscloudformation |
| Api      | somenamegraphql  | Create    | awscloudformation |
| Hosting  | somename    | Create    | awscloudformation |

amplify push
amplify publish -c

----------- ALL WELL TILL HERE ------------

amplify env add
<< choose the aws profile of the account from another organization>>

Current Environment: envtwo

| Category | Resource name      | Operation | Provider plugin   |
| -------- | ------------------ | --------- | ----------------- |
| Auth     | somename00000000 | Create    | awscloudformation |
| Storage  | somenamestorage  | Create    | awscloudformation |
| Api      | somenamegraphql  | Create    | awscloudformation |
| Hosting  | somename    | Create    | awscloudformation |

amplify push gives this error :

ISSUE # 1 :
CREATE_FAILED      HostedUIProvidersCustomResourceInputs Custom::LambdaCallout Mon Mar 22 2021 19:33:12 GMT-0400 (Eastern Daylight Time) Failed to create resource. See the details in CloudWatch Log Stream: 2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4

From Log stream :
2021-03-22T23:33:10.456Z	f141c0f0-b781-433a-bdcd-aa5122b4cb5d	INFO	Response body:
{
    "Status": "FAILED",
    "Reason": "See the details in CloudWatch Log Stream: 2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4",
    "PhysicalResourceId": "2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4",
    "StackId": "arn:aws:cloudformation:us-east-1:[ACCOUNT_NUMBER]:stack/amplify-someapplication-envtwo-192949-authsomeapplication10ec9a3f-N3HNQXJZ0DNP/bcf0b9f0-8b66-11eb-b644-0a8082d84415",
    "RequestId": "5b4f5ad6-5342-4bd7-8984-638cdf17bdf1",
    "LogicalResourceId": "HostedUIProvidersCustomResourceInputs",
    "NoEcho": false,
    "Data": {
        "err": {
            "message": "client_id, client_secret and authorize_scopes are all required idp details.",
            "code": "InvalidParameterException",
            "time": "2021-03-22T23:33:10.414Z",
            "requestId": "b5cef6fd-40d0-474a-9005-84cacf7c8f51",
            "statusCode": 400,
            "retryable": false,
            "retryDelay": 87.55256440291616
        }
    }
}

ISSUE # 2. : After this, canNOT even remove this env locally !!!!!

amplify env remove envtwo

? Are you sure you want to continue? This CANNOT be undone. (This will delete 'envtwo' environment of the project from the cloud) Yes
⠋ Deleting resources from the cloud. This may take a few minutes...
Deleting env:envtwo
⠧ Deleting resources from the cloud. This may take a few minutes...
Error in deleting env:envtwo
null
✖ remove env failed: null
Forbidden:
    at Request.extractError (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/services/s3.js:692:35)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at callNextListener (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:96:12)
    at IncomingMessage.onEnd (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/event_listeners.js:313:13)
    at IncomingMessage.emit (node:events:381:22)
    at IncomingMessage.emit (node:domain:470:12)
    at endReadableNT (node:internal/streams/readable:1307:12)
    at processTicksAndRejections (node:internal/process/task_queues:81:21)

GraphQL schema(s)

# Put schemas below this line

Log output

From Log stream :
2021-03-22T23:33:10.456Z	f141c0f0-b781-433a-bdcd-aa5122b4cb5d	INFO	Response body:

{
    "Status": "FAILED",
    "Reason": "See the details in CloudWatch Log Stream: 2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4",
    "PhysicalResourceId": "2021/03/22/[$LATEST]6ad1186de94e49099861fa96da2816b4",
    "StackId": "arn:aws:cloudformation:us-east-1:[ACCOUNT_NUMBER]:stack/amplify-someapplication-envtwo-192949-authsomeapplication10ec9a3f-N3HNQXJZ0DNP/bcf0b9f0-8b66-11eb-b644-0a8082d84415",
    "RequestId": "5b4f5ad6-5342-4bd7-8984-638cdf17bdf1",
    "LogicalResourceId": "HostedUIProvidersCustomResourceInputs",
    "NoEcho": false,
    "Data": {
        "err": {
            "message": "client_id, client_secret and authorize_scopes are all required idp details.",
            "code": "InvalidParameterException",
            "time": "2021-03-22T23:33:10.414Z",
            "requestId": "b5cef6fd-40d0-474a-9005-84cacf7c8f51",
            "statusCode": 400,
            "retryable": false,
            "retryDelay": 87.55256440291616
        }
    }
}

When you add a new env, did you get a question around entering your client id and client secrets for the auth resource in your nw environment?

Nope - NOT prompted

@bishonbopanna Can you update your credentials by running amplify update auth  to get you unblocked we working on the resolution

@bishonbopanna Specifically you can go through the following flow to unblock yourself while we push out a fix for this:
kaustavg@3c22fb229ff6 authbug % amplify update auth
Please note that certain attributes may not be overwritten if you choose to use defaults settings.
Using service: Cognito, provided by: awscloudformation
 What do you want to do? Update OAuth social providers
 Select the identity providers you want to configure for your user pool: Facebook

 You've opted to allow users to authenticate via Facebook.  If you haven't already, you'll need to go to https://developers.facebook.com and create an App ID.

 Enter your Facebook App ID for your OAuth flow:  yourappid
 Enter your Facebook App Secret for your OAuth flow: yoursecret
Successfully updated auth resource authbugb417c8b2 locally

@ammarkarachi , @kaustavghosh06  - thankyou, the workaround did the job. Any recommendation on removing the failed envs as per issue # 2 ?

the workaround : amplify update auth has addressed the removal of the env issue # 2 as well.

@bishonbopanna The #2 issue seems unrelated to auth and seems like its tied to the creds used. If you update the aws profile of the current env to that of envtwo and then try removing envtwo, you should be able to get around it.

@bishonbopanna We've released a fix for the issue with add auth -> push which was your original issue. Closing this issue.

thanks @kaustavghosh06 - how to get the fix ? Is there a need to pull the latest version ? also issue # 2 is not resolved as per your recommendation - Please let me know if you need any other info.

@bishonbopanna #2 happens due to wrong access/creds. What does your amplify/.config/local-aws-info.json look like? Did you make the necessary changes based on my recommendation above?
You need the latest version of the CLI to get a fix for #1 although at this point I believe you're past that issue based on the workaround provided yesterday,

thanks @kaustavghosh06, will pull the latest, yes work around is in use.
Yes I did as per your recommendation. here are the contents
{   "envone": {     "configLevel": "project",     "useProfile": true,     "profileName": "something-beta-env"   },   "envtwo": {     "configLevel": "project",     "useProfile": true,     "profileName": "something-beta-env"   },   "envthree": {     "configLevel": "project",     "useProfile": true,     "profileName": "something-us-east-1"   } }

@bishonbopanna Are you trying to remove envone from evntwo or envthree?

@kaustavghosh06 - envtwo

Okay, so the CloudFormation stack has already deleted and the corresponding resources are deleted  along with he Amplify backend environment in the console. You would need to remove the environment from environment from the "team-provider-info.json` file locally to get rid of it from showing locally.

ok will do, thanks again @kaustavghosh06


Fix server-side pagination

When the last page from new props is smaller than the current page in the page box, the current page needs to be the last page. This happens when new datasets from the server side get smaller than previous ones by using search boxes.

Is there any plan for when this update will be published?

Bumping as it's been a while.
@tannerlinsley are you planning to publish this fix or will there be no new releases of the v6 branch?


Fix TOOLSET builds

Resolves #137 by updating the --TOOLSET flag to --toolset which matches node-pre-gyp's expectations. More background in #137
Confirmed versions are publishing successfully over at https://travis-ci.org/mapbox/node-cpp-skel/builds/388725890
cc @millzpaugh @springmeyer @GretaCB @allieoop

goodness, thanks for catching this subtle cause of failure. I must have done a find and replace that was not case sensitive and should have been. Thank you.


[BUG]LNURLp / LNaddress not working in v0.12.12 in combi with lnurlp extension v0.5.6

I am running LNBits on v0.12.12 with LNURLp extension v0.5.6.
The Lightning address seems to be broken. I can scan the lightning adress from LNBits PayLinks extension and pay to it but :
<EMAIL_ADDRESS>returns an "address not existing". Decoder confirms issue, too.
Not sue if this is a LNBits v0.12.12 issue, as with earlier versions my LNaddresses worked just fine.
I have tried this:
https://lnbits.btc-payserver.eu/lnurlp/api/v1/well-known/testuserwallet
returns propper json:
{
"tag": "payRequest",
"callback": "https://lnbits.btc-payserver.eu/lnurlp/api/v1/lnurl/cb/FGgzTb",
"minSendable": 1000,
"maxSendable": 100000,
"metadata": "[["text/plain", "Payment to testuser"], ["text/identifier"<EMAIL_ADDRESS>"commentAllowed": 200,
"allowsNostr": true,
"nostrPubkey": "e1585e0db30c182c52c6fd18a2b1b0fdb4ee73ddfdee873d23aaf77977310e4d"
}

https://lnbits.btc-payserver.eu/.well-known/lnurlp/testuserwallet
returns error:
{
"status": "ERROR",
"reason": "Address not found."
}

There seem to be more people having the same issue as discussed in LNBits Telegram Group.
Thanks for looking into this. It would be nice if it could get a quick fix.
Greets

Can you please confirm that it is not this issue: https://github.com/lnbits/lnbits/issues/2540#issuecomment-2260544749

I believe the issue is caused by a conflict between lnaddress and lnurlp in mapping the same route:
* lunrlp: https://github.com/lnbits/lnurlp/blob/main/__init__.py#L20

* lnaddress: https://github.com/lnbits/lnaddress/blob/main/__init__.py#L21

There is no deterministic way in which the extensions are loaded, therefore the unpredictable behavior.
Suggestion: remove lnaddress.

Note: the extension conflict has been handled in v1

Yes, the conflict between lnaddress and lnurlp was the issue. Had to uninstall lnaddress extension to make lnurlp work.


Report and discard bib records with same legacy IDs and 001:s as previously transformed records

@fontanka16 Does the description say that 001:s should be reported and discarded even if they are not used as legacy ID?
Scenarios;
997 is legacy_id and duplicated, 001 is also duplicated*
legacy_id_from_997: 123
997: 123
001: 456

legacy_id_from_997: 123
997: 123
001: 456

001 is legacy_id and is duplicated
legacy_id_from_001: 456
001: 456

legacy_id_from_001: 456
001: 456

997 is legacy_id and duplicated, 001 is not duplicated
legacy_id_from_997: 123
997: 123
001: 83974

legacy_id_from_997: 123
997: 123
001: 15648

See change to name -- this only concerns legacy id:s.


fix(common): live-9252 update buy/sell api and expose methods in onRampProvider

📝 Description
The onRampProvider was using out of date cached CDN endpoint, which hid available currencies like NEAR.
This PR updates the API that informs whether currencies can be bought or sold so that new currencies like NEAR show as available.

Expose isCurrencySupported and getSupportedCurrencyIds methods from useRampProvider hook.
Add unit tests for helper functions

❓ Context

Impacted projects: ledger-live-common ledger-live-desktop ledger-live-mobile -
Linked resource(s):  https://ledgerhq.atlassian.net/browse/LIVE-9252?atlOrigin=eyJpIjoiNTJjMGU2Mjc4NGIwNGNiNzhkZmU4ZmFjMWZjYTYyMjMiLCJwIjoiaiJ9

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

/generate-screenshots

/generate-screenshots

/generate-screenshots

as there are a screenshot conflict, please rebase and /generate-screenshots when the CI is stable on develop again

/generate-screenshots


use tabs for jsx

This allows for the use of tabs on JSX files.
Also, while adding this fix, I saw the jsx-quotes property, but I don't see it anywhere here https://github.com/feross/eslint-config-standard-jsx/blob/master/eslintrc.json. Would it be a deprecated property?

I think that #19 fixes this stuff along with the updated versions.  Once it is ready to be published I will update back here with confirmation.

I believe this is fixed, please feel free to re-open if not.


Add documentation for pyenv plugin

Partially addresses #7175
Adds a README for the pyenv plugin. Plugin already has documentation on the wiki: https://github.com/robbyrussell/oh-my-zsh/wiki/Plugins#pyenv

Thanks!


Launcher v3.0.3 Update Error

Release Type: Official Release/GitHub (please choose appropriate option)
*Version: v3.0.3
*Platform: Windows
This version will not update to the latest version. (Tried running as admin and ran with firewall setting off in case there was any connection issues).
Reproduce (Unknown)
Did try reinstalling xenko at the main website.
Screenshot

Can you try using my different Xenko distribution method?
https://github.com/phr00t/xenko/releases/tag/3.1b-R1
Please mention me with results if you try it!

Lacks detail, issue either fixed or out of date given the changes nuget has gone through, I'll close this off.


Enable tests in the explorer

Resolves a known issue per @natemcmaster.

@rjmooney,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@rjmooney, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot


[SpotBugs][pulsar-io]Enable spotbugs for pulsar-io-hdfs2 and pulsar-io-hdfs3

Master Issue: #8843
Motivation
Enable spotbugs for the pulsar-io module.
Modifications
Enable soptbugs for pulsar-io-hdfs2 and pulsar-io-hdfs3
Verifying this change
mvn clean install -pl pulsar-io/hdfs2
mvn clean install -pl pulsar-io/hdfs3

@fxbing seems that this PR does not need to update docs?
P.S. when submitting a PR, can you provide a doc label (tick the box) in the PR template contains info about doc? This helps others know more about the changes. Thanks


Peer to Peer code review:

Hi @evansnyamekye,
You had a perfect job implementing the first capstone project. Still, I will suggest one thing use semantic tags instead of div and <a> you can use <ul> and <li> tags.
✅ Thank you for distributing your project.
You did an excellent job 😄

Hi @tajulafreen ...Thank you for the review. 😀 👍


Silent Notification not working

I'm using version 0.8.12 from F-Droid on LineageOS 15.1 without GApps installed.

As you can see from the screenshot I am also using Conversations and Syncthing both with the "always active" notification (to avoid app to be killed by Android and to keep receiving notifications), but both those apps have the silent notification working correctly, while Riot notification is always in foreground (when active), and it's quite annoying (it's also visible in the lock screen).
For what I remember, before updating from LOS 14.1 (and Riot 0.8.7) to LOS 15.1 it was working properly, but I may be wrong.

I'm closing this issue as it turned out to be a different way of LOS 15.1 (Android 8.1) of handling notifications compared to LOS 14.1 (Android 7.1).
In particular you can change the notification behavior by selecting Listening for events in the app notification settings, as shown below

and then changing Importance from Medium to Low

As you can see below the notification now is not in foreground anymore

Maybe you could do something like making this setting the default one, I don't know.
By the way I believe this issue can be considered solved.


getNewTranslation() executes a useless query

Survey model:
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\SoftDeletes;
use Illuminate\Database\Eloquent\Model;

class Survey extends Model
{
    use SoftDeletes, \Dimsav\Translatable\Translatable;

    public $translatedAttributes = ['name'];

    protected $fillable = [
        'start_at',
        'end_at',
    ];
}

SurveyTranslation model:
<?php

namespace App\Models;
use Illuminate\Database\Eloquent\Model;

class SurveyTranslation extends Model
{
    public $timestamps = false;

    protected $fillable = [
        'name',
        'locale',
    ];
}

Test route:
        Route::get('ttest', function() {
            $s = new \App\Models\Survey;
            $s->getNewTranslation('en');
        });

This should execute 0 queries, but runs this one instead:
select * from `survey_translations`
where `survey_translations`.`survey_id` is null
and `survey_translations`.`survey_id` is not null

Apparently, this won't be executed if I interrupt execution before 'add' is called on the translations collection here:
https://github.com/dimsav/laravel-translatable/blob/master/src/Translatable/Translatable.php#L425
I don't understand what's happening...

@dimsav ?

@koichirose
public function getNewTranslation($locale)
{
$modelName = $this->getTranslationModelName();
$translation = new $modelName();
$translation->setAttribute($this->getLocaleKey(), $locale);
$this->translations->add($translation);
    return $translation;
}

Have you seen $this->translations->add($translation); ?
it's because $this->translations <=> ($this->translations())->add($translation);
Have you got it? $this->translations() will issue a query for the associated survey_translations table.

I don't understand your question.
That code is from this package itself.
I have:
this->translations->add($translation);

Anyone?

Hey mate, sorry for replying with delay.
What makes this extra query is not the add() method, but $this->translations.
$this->translations is run by laravel magic and if you debug a little bit you'll see that it fetches the existing relationships.
Maybe there is a workaround for this. Feel free to recommend any changes.

Hmm I have no idea.
Since you need to add the newly created translation to the existing $this->translations collection, it doesn't look like there could be a workaround for this.
I'll try to limit the usage of this as much as possible.


Update composer.json

Let me know if you want some basic instructions for how to install via composer.
It would be along the lines of...
"repositories": [
{
"type": "git",
"url": "URL to the GIT Repository"
}
]
Then
composer require 'imsglobal/lti-1p3-tool'

There is a branch with composer https://github.com/IMSGlobal/lti-1-3-php-library/tree/composer if you want to use that, i haven't merged it into master as it breaks for people not using composer


[Feature Request]: Create sharable embed of profile cards

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
We need to provide users a sharable card which they can embed in their websites or in their github profiles.

[ ] Design an attractive card
[ ] Allow creating embeds
[ ] Allow sharing feature

Use Case
It would increase visibility of git-re and also provide users a dynamic embed card.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[ ] I'm a GSSOC'24 contributor
[X] I have starred the repository

Thank you for creating this issue! 🎉 We'll look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊
You can also check our CONTRIBUTING.md for guidelines on contributing to this project.

can you assign me this issue?
@ashutosh-rath02

Hey @ashutosh-rath02, apologies for the issues taken and not completed. Got some important college work, if you remember me :).
Back now to work on some of open-source projects, found you repo the best option now also. Can you assign this issue to me, would love to continue contributing to the Git-Re.

Go ahead @NilanchalaPanda and welcome back

This issue has been automatically closed because it has been inactive for more than 30 days. If you believe this is still relevant, feel free to reopen it or create a new one. Thank you!

Hello @ashutosh-rath02! Your issue #338 has been closed. Thank you for your contribution!


Eliminate usage of gen-region.sh

Services can now retrieve the region from the miner directly via hm_pyhelper.miner_json_rpc.get_region. There is no longer a need for gen-region.sh to write the region to file.
Acceptance criteria

hm-miner: Remove gen-region.sh and all references to it.
hm-pyhelper: Update logic in hm_pyhelper.miner_param.retry_get_region to invoke hm_pyhelper.miner_json_rpc.get_region instead of reading from file.
hm-pktfwd: Remove references to REGION_FILEPATH in hm-pktfwd.
Double check there are no other services that depend on the region file. At the time of writing, only hm-pktfwd uses this method.

Only problem with this, is that JSON RPC is unreliable...
https://github.com/NebraLtd/hm-diag/issues/195


Pipelining behaviour question

Hello,
I am trying to understand how does the pipelining behavior works. My expectation was that it would create one connection for every 5 requests (due to https://curl.haxx.se/libcurl/c/CURLMOPT_MAX_PIPELINE_LENGTH.html) when using this option.
But then I run a tcp dump to check what was going on and it was one connection for every request.
How is the pipelining behavior supposed to work ?
def request(pipelining = true)
  hydra = Typhoeus::Hydra.new(pipelining: pipelining)
  10.times do
    url = 'http://google.com'
    request = Typhoeus::Request.new(url, params: { q: "test" })
    hydra.queue(request)
  end
  hydra.run
end
request(true)

tcpdump
root@e09bf0b135b8:/app# tcpdump -i eth0 -w tcpdump-google.out -s 1520 port 80
tcpdump: listening on eth0, link-type EN10MB (Ethernet), capture size 1520 bytes
^C70 packets captured
70 packets received by filter
0 packets dropped by kernel

root@e09bf0b135b8:/app# tcptrace  tcpdump-google.out
1 arg remaining, starting with 'tcpdump-google.out'
Ostermann's tcptrace -- version 6.6.7 -- Thu Nov  4, 2004

70 packets seen, 70 TCP packets traced
elapsed wallclock time: 0:00:00.005269, 13285 pkts/sec analyzed
trace file elapsed time: 0:00:00.203851
TCP connection info:
  1: e09bf0b135b8:54578 - sin01s16-in-f14.1e100.net:80 (a2b)    4>    3<
  2: e09bf0b135b8:54580 - sin01s16-in-f14.1e100.net:80 (c2d)    4>    3<
  3: e09bf0b135b8:54582 - sin01s16-in-f14.1e100.net:80 (e2f)    4>    3<
  4: e09bf0b135b8:54584 - sin01s16-in-f14.1e100.net:80 (g2h)    4>    3<
  5: e09bf0b135b8:54586 - sin01s16-in-f14.1e100.net:80 (i2j)    4>    3<
  6: e09bf0b135b8:54588 - sin01s16-in-f14.1e100.net:80 (k2l)    4>    3<
  7: e09bf0b135b8:54590 - sin01s16-in-f14.1e100.net:80 (m2n)    4>    3<
  8: e09bf0b135b8:54592 - sin01s16-in-f14.1e100.net:80 (o2p)    4>    3<
  9: e09bf0b135b8:54594 - sin01s16-in-f14.1e100.net:80 (q2r)    4>    3<
 10: e09bf0b135b8:54596 - sin01s16-in-f14.1e100.net:80 (s2t)    4>    3<

When I use the CLI, it works:
root@e09bf0b135b8:/app# curl http://www.google.com http://www.google.com

root@e09bf0b135b8:/app# tcpdump -i eth0 -w tcpdump-google-curl.out -s 1520 port 80
tcpdump: listening on eth0, link-type EN10MB (Ethernet), capture size 1520 bytes
^C14 packets captured
14 packets received by filter
0 packets dropped by kernel

root@e09bf0b135b8:/app# tcptrace tcpdump-google-curl.out
1 arg remaining, starting with 'tcpdump-google-curl.out'
Ostermann's tcptrace -- version 6.6.7 -- Thu Nov  4, 2004

14 packets seen, 14 TCP packets traced
elapsed wallclock time: 0:00:00.004786, 2925 pkts/sec analyzed
trace file elapsed time: 0:00:00.099330
TCP connection info:
  1: e09bf0b135b8:34002 - syd15s02-in-f36.1e100.net:80 (a2b)    7>    7<  (complete)

I've never used that option before but I think you are passing the wrong key to Hydra. Ethon::Curls::Options contain all possible options and it's usually the curl options but lowercased plus with the CURLMOPT I think. So in that case you might need to pass https://github.com/typhoeus/ethon/blob/4c820e2116c7dba030e84d576a0592afd430eba1/lib/ethon/curls/options.rb#L219?

OK just saw the pipelining option https://github.com/typhoeus/ethon/blob/4c820e2116c7dba030e84d576a0592afd430eba1/lib/ethon/curls/options.rb#L214 so ignore what I said above

Be aware that pipelining is unreliable unless CURLOPT_PIPEWAIT is set to true (sometimes it works, sometimes you get multiple connections). See the Multiplexing section of https://curl.haxx.se/docs/http2.html. As far as I can tell, Ethon does not support this yet.


PSP specific implementation of getAvailablePaymentMethods method

method is PSP specific and should first only be implemented for PayOne
it should load the PaymentMethodInfo objects from a simple config file located within the classpath of the project

the config file musst contain following information:

list of enabled method IDs
config values per method
config value payment interface (inherited from the PSP config)
config value method
config value name (localization possible, but only english provided by default)

The first quick implementation will reload the values every time the method is called. Caching will be a future task.

implementation successfully tested within coeur shop


Iteratively adding constraints to a large model

Hi,
I was trying to use good-lp to solve a mixed integer linear programing problem. Is it possible to iteratively adding constraints to a model? If yes, is there an example how to use the add_constraints() function?
Thanks,
Hang

The function you are looking for is probably https://docs.rs/good_lp/latest/good_lp/solvers/trait.SolverModel.html#method.with
there are many examples in this repo and in the documentation


Fix model translation in Dymola 2023

Dymola 2023 seems to throw errors when loading dsin files. This needs further investigation and fix.
Example
 File "...\ebcpy\ebcpy\modelica\manipulate_ds.py", line 48, in convert_ds_file_to_dataframe
    size_initial_names = int(content[number_line_initial_name].split("(")[-1].split(",")[0])
ValueError: invalid literal for int() with base 10: 'multizone.zoneParam[1].AExt[3]'

A temporary fix is to disable this feature in the DymolaAPI by using: extract_variables=False as a keyword argument.


Add a tanzu apps config hint to local-source workload commands

Description of problem
To streamline common commands the CLI introduced a configurable default for flags that are often reused with the same values.
Because the capability is new, there is an opportunity to provide hints when users run commands with values that could be saved.
Proposed solution
<Please describe your proposed solution, preferably in the following style:>
Given I create/apply a workload using --local-path AND --source-image
When the CLI returns feedback (the diff and confirmation)
Then I should see an information message letting me know about the config command

Example
$ tanzu apps workload create NAME --local-path <bla> --source-image <bla>
Create workload:
      1 + |---
      2 + |apiVersion: carto.run/v1alpha1
      3 + |kind: Workload
      4 + |metadata:
      5 + |  labels:
The source-image can be configured via “tanzu apps config”
create (y/N)

Concern: If we establish a pattern of hinting for each flag, the notice section could get pretty crowded, and if we hint on every command run, we risk feeling like spam, resulting in alert fatigue / being ignored.
$ tanzu apps workload create NAME --local-path <bla> --source-image <bla>
Create workload:
      1 + |---
      2 + |apiVersion: carto.run/v1alpha1
      3 + |kind: Workload
      4 + |metadata:
      5 + |  labels:

The following defaults were applied: “debug:false”
The source-repo can be configured via “tanzu apps config”
These defaults can be changed via “tanzu apps config”

create (y/N)

Describe alternatives you've considered
We could rely on the feedback message to give a hint, rather than calling out one field specifically.
The following would both let someone know what we did (what defaults) and what to do about it (apps config) - if they run apps config, they will see all the configurable values. - it is less direct, but it does reduce the number of informational messages.
The following defaults were applied: “debug:false”
Defaults for flag values can be set via “tanzu apps config”

Additional context
Add any other context or screenshots about the feature request here.

Closing this as we decided not to implement #108


[NOT A BUG] Improving documentation about standalone

Hello,
I am currently deploying a nuitka built standalone program on a couple of different machines, and noticed that the api-ms-win-crt-*.dll files that come with Nuitka aren't enough depending on the target OS.
Example:
My program asked for api-ms-win-crt-conio-l1-1-0.dll which wasn't present on a vanilla Windows 7 32-bit OS, and which wasn't bundled by nuitka when compiled on Win10 x64.
Indeed, Windows 10 does ucrt.dll, and rehooks all api-ms-win-crt* dll calls to that specific DLL.
I came to the conclusion that in order to get my standalone executable work on all Windows platforms from Win7 to Win10, I had to install Visual C 2015 redistributable first, vc_redistx.86.exe or vc_redist.x64.exe, which has all the necessary updates to use ucrt.dll from WinXP to Win10.
Indeed the version of Visual C (2012/2014/2015) changes depending on the Python version used by Nuitka.
I found that neat table here https://wiki.python.org/moin/WindowsCompilers which I updated with the year

Visual C++ verison
Year
CPython

14.X
2015
3.5, 3.6, 3.7, 3.8

10.0
2010
3.3, 3.4

9.0
2008
2.6, 2.7, 3.0, 3.1, 3.2

I think it would be nice to state that in the Nuitka documentation somewhere.
Before making a PR, can you confirm my findings ?
Best regards.

Windows 7 is not really on the list of things intended to be supported, and I wouldn't be too surprised if many Python extension modules were no longer running on it. But if I understand you correctly, this is what it takes, I guess we should have a HowTo document.
One important element of howto on Linux, is to build on the oldest system you intend to support. Going forward is usually possible, going backware, the way programs on Linux e.g. link against DLL requires higher versions of the libc, which then will only work with a certain kernel.
But if this works for Windows 10 to 7, surely we can document it. I just fear that's a moving target. And now that Windows 7 is so deeply EOL it's rather late to start supporting it.
Also notice that Nuitka works with every MSVC with even Python 2.6, and every new enough MinGW 64 for every Python.
We now ignore everything in the system folder with good success. If we open this up for the ms-api libraries, we need to know really well what we are doing though.
Scary to me, but if it is tested and works, right. But do we maintain it?

Well yes, Windows 7 is EOL, but this also applies to Windows 8 and 8.1.
Python already ships with 2 standard VC dlls, vcruntimeXXX.dll and mfcuXXX.dll.
The thing is that vc_redist is already baked into Windows 10 (unless a newer python version will be linked with a newer vc_redist which isn't yet baked in).
It's just giving an advice that installing vc_redist on elder Windows versions will most probably fix some missing dlls that don't ship with a Nuitka standalone program.
There's no maintenace burden to me ;)
As of my understandings, one may compile an executable with any windows version for any windows other windows version (excluding windows 9X ;)), as long as one respects the bitness and ships the right runtime dlls.

As for the tests, I'm currently rolling out a Nuitka compiled program on about 2500 computers, which range from Windows 7 to Windows 10, Windows Server 2008 R2 to Windows Server 2019, all in x86 and x64 bits.
I'd like to get rid of Windows 7 in most cases too, but well... industrial computers can't be upgraded sometimes.
My first tests, about 100 machines, went good.
Most machines had already the vc dlls packed by another app, but since I packaged vc_redist into my installer, I didn't find any machine that could not run my program :)

Closing this issue since it has been resolved.


fix: fvm_ipld_encoding, forest_db, and ethers dependencies

Why
Because we want ursa to compile!
There are 2 things wrong and one potential issue:

The current build issue with selecting a version for fvm_ipld_encoding
The latest forest_db commit breaks our dependence on the Store trait's delete fn and its implementation for rocksdb. We're getting this breaking change because the git dependency is unpinned to a revision and they removed the fn (ChainSafe/forest#2638)
Noticed that ethers-rs was unpinned, and a breaking change there could cause more issues, and also that ursa-app is not using the workspace dep

What

bump fvm_ipld_encoding to 0.3.3
allow depreciated use of the Cbor trait in ursa-index-provider, because ipld_encoding 0.3.3 causes clippy warnings. We require our own implementation using the trait directly because the new pattern (to_vec) breaks integrations
pin forest_db to a commit with delete still in the Store trait and implemented for RocksDB
use crate release 1.0.2 for ethers-rs
ursa-app uses workspace ethers version

Notes
We can update to the new garbage collection method from forest_db in a future PR
Checklist

[x] I have made corresponding changes to the tests
[x] I have made corresponding changes to the documentation
[x] I have run the app using my feature and ensured that no functionality is broken

We shouldn’t be using deprecated methods


Text editor layer always on the top when function parameter hints shows up

Operating system or device - Godot version:
OS:Windows 10 ,OS X
Godot Version : v2.1.1.stable.official
Issue description (what happened, and what was expected):
As title described.
Steps to reproduce:

Create a node
Add a script to the node just created
edit the script and write some function which requires parameter but don't finish it
open a new dialog ,like Scene->Project Settings.

And here is the screenshot:

Duplicate of #7156 -> #6769


refactor ipemu probes

[x] refactor probe signatures to improve readability
[x] refactor to an single Probe to avoid using Record on the [[T1Interface]].

We should adopt layer in a near future.


[Addition] Canary (1.18.2 + 1.19.2)

Checklist

[X] I've checked that my mod suggestion hasn't already been added to one of the lists.
[X] I've checked the latest issues to make sure that this mod hasn't already been suggested.

What is your mod's name?
Canary
What is your mod's main version?
1.18.X (Forge)
Input a link to your suggested mod. Make sure it isn't hosted on a site listed on StopModReposts.
https://www.curseforge.com/minecraft/mc-mods/canary
What is your reasoning for including this mod?
It is a fork of Lithium with patches backported from the 1.19 version to the 1.18 version. It also fixes many of the issues that Radium Reforged hasn't addressed.
The author also says that in the future they want to add their own performance improvements not found in Lithium/Radium Reforged.
(OPTIONAL) Give some extra information about the mod.
No response

One quirk is that the mod just went closed source.

One quirk is that the mod just went closed source.

He claims that its only temporary until he's finished but I do agree that it's odd

One quirk is that the mod just went closed source.

He claims that its only temporary until he's finished but I do agree that it's odd

If he makes another release while the code is still closed source, that would violate Lithium's license.
But if its just temporary till the next release... thats still odd, but fine.

It's back to being open source


[docs] Fix the migration guide doc page

In this pull request, we are applying some markdown fixes to the migration guide page in the docs folder.
There is no change in the content of the page, only markdown fixes, as you can see in the screenshots below:
Migration guide before:

Migration guide after:

@nicolasiensen Thank you!


Quiche changes to avoid gcc flags on Windows

Description:
The -w flags are not recognized by msvc.
Signed-off-by: Yechiel Kalmenson<EMAIL_ADDRESS>Signed-off-by: William Rowe<EMAIL_ADDRESS>Risk Level: Low
Testing: Local on Windows
Docs Changes: None
Release Notes: None

In PR 8280 we proposed to accept all invalid, unguarded gcc #pragma statements with the /Wd4068 toggle. This gets quiche building if the command line is not polluted, as proposed in this patch.

There are a few flags which occur in the test/ framework for quiche, we are attaching these to this ticket as a second commit to this PR.


feat: added healthz endpoint

AsIs
Currently no endpoint in app to check application status, and we could not implement probes in kubernetes manifests.
ToDo

[x] implement /healthz to express router
[x] update application manifest

(Optional) Acceptance criteria
Confirm readinessProbes are success in kubectl logs
(Optional) Additional contexts
N/A

Confirmed that readinessProbe have been successfully completed after applying manifests in PR.
24-03-03 14:50:20 git/6ow3idGirl [main] % kubectl apply -f manifests/express.yaml
deployment.apps/express configured
service/express unchanged

# note that application have used latest image hash in GHCR after published with merge of PR
# sha256:3415be2acda5293c6e562762d972e67df4e1c568b4c8cb88cf183eed15b7f97c
24-03-03 14:51:41 git/6ow3idGirl [main] % kubectl -n 6ow3idgirl describe pod express-698ddbcc75-6hwt4
Name:             express-698ddbcc75-6hwt4
Namespace:        6ow3idgirl
Priority:         0
Service Account:  default
Node:             gke-playgrounds-default-pool-dfe67b64-nsd5/<IP_ADDRESS>
Start Time:       Sun, 03 Mar 2024 14:50:36 +0900
Labels:           app=lowmidgirl
                  pod-template-hash=698ddbcc75
Annotations:      kubectl.kubernetes.io/restartedAt: 2024-03-01T22:18:04+09:00
Status:           Running
IP:               <IP_ADDRESS>
IPs:
  IP:           <IP_ADDRESS>
Controlled By:  ReplicaSet/express-698ddbcc75
Containers:
  express:
    Container ID:   containerd://1a48a74a9f344a9184f0f3f1fa4c58649a6b0bee62cc36e8413125f78adc0d2b
    Image:          ghcr.io/hwakabh/6ow3idgirl:latest
    Image ID:       ghcr.io/hwakabh/6ow3idgirl@sha256:3415be2acda5293c6e562762d972e67df4e1c568b4c8cb88cf183eed15b7f97c
    Port:           <none>
    Host Port:      <none>
    State:          Running
      Started:      Sun, 03 Mar 2024 14:50:58 +0900
    Ready:          True
    Restart Count:  0
    Readiness:      http-get http://:8080/healthz delay=20s timeout=10s period=60s #success=1 #failure=3
    Environment:    <none>
    Mounts:
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-62ttq (ro)
Conditions:
  Type              Status
  Initialized       True
  Ready             True
  ContainersReady   True
  PodScheduled      True
Volumes:
  kube-api-access-62ttq:
    Type:                    Projected (a volume that contains injected data from multiple sources)
    TokenExpirationSeconds:  3607
    ConfigMapName:           kube-root-ca.crt
    ConfigMapOptional:       <nil>
    DownwardAPI:             true
QoS Class:                   BestEffort
Node-Selectors:              <none>
Tolerations:                 node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                             node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:
  Type    Reason     Age   From               Message
  ----    ------     ----  ----               -------
  Normal  Scheduled  77s   default-scheduler  Successfully assigned 6ow3idgirl/express-698ddbcc75-6hwt4 to gke-playgrounds-default-pool-dfe67b64-nsd5
  Normal  Pulling    75s   kubelet            Pulling image "ghcr.io/hwakabh/6ow3idgirl:latest"
  Normal  Pulled     57s   kubelet            Successfully pulled image "ghcr.io/hwakabh/6ow3idgirl:latest" in 17.880556419s (17.880593989s including waiting)
  Normal  Created    56s   kubelet            Created container express
  Normal  Started    55s   kubelet            Started container express

# logs of application side with readinessProbes
24-03-03 14:52:22 git/6ow3idGirl [main] % kubectl -n 6ow3idgirl logs express-698ddbcc75-6hwt4

><EMAIL_ADDRESS>start
> node ./bin/www

Express app started with listening on <IP_ADDRESS>:8080 ...
(node:22) Warning: Accessing non-existent property 'route' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
GET /healthz 200 6.637 ms - 15

# and all the pod had been updated with rollouts
24-03-03 14:54:52 git/6ow3idGirl [main] % kubectl -n 6ow3idgirl get pod
NAME                       READY   STATUS    RESTARTS   AGE
express-698ddbcc75-6hwt4   1/1     Running   0          4m18s
express-698ddbcc75-ckxkt   1/1     Running   0          3m17s
express-698ddbcc75-rwpr8   1/1     Running   0          74s


Modifications to support single-manager local bootstrap

Fixes #1190
$ platform/bin/deploy

@ndegory @bquenin Thanks for the review and updates.


major: update awilix dependency to v4

Update all packages, most notably Awilix itself

BREAKING CHANGE — changes required Awilix version.
No code-specific changes.

Coverage remained the same at 100.0% when pulling e7c0341a447baaacb403210d3880cbaef69c0b56 on jeffijoe:awilix-v4 into ff3afebf230f402d72a79fb9263143297429841e on talyssonoc:master.

Coverage remained the same at 100.0% when pulling e7c0341a447baaacb403210d3880cbaef69c0b56 on jeffijoe:awilix-v4 into ff3afebf230f402d72a79fb9263143297429841e on talyssonoc:master.


can't get wish history

ehh kindda weird, how can I fix it ?

Made a small section about this problem in the readme : https://github.com/thesadru/genshinstats#why-doesnt-get_wish_history-return-a-normal-list
Please use something like
for wish in gs.get_wish_history(...):
    print(wish)

Made a small section about this problem in the readme : https://github.com/thesadru/genshinstats#why-doesnt-get_wish_history-return-a-normal-list
Please use something like
for wish in gs.get_wish_history(...):
    print(wish)

oh, thanks, my bad:)


[theme] Declaring body1 in createTheme resets font family

Duplicates

[x] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
If body1 is redeclared as documented for responsive font sizes, the font-family is reseted.
Expected behavior 🤔
Redeclaring body1 will not reset the font styles and everything from allVariants block will be applied.
Steps to reproduce 🕹
Steps:

createTheme with a custom font-family in  the typography.allVariants block
redeclare the typography.body1 as described in the documentation with new fontSizes

const theme = createTheme({
 ...
 typography: (palette) => ({
      allVariants: {
        fontFamily: ['OurOwnFont', 'serif'].join(','),
        color: palette.text.primary,
        fontWeight: 500,
        lineHeight: 1.25,
      },
    }),
});

theme.typography = {
    ...theme.typography,
    body1: {
      fontSize: theme.typography.pxToRem(16),
      [theme.breakpoints.down('md')]: {
        fontSize: theme.typography.pxToRem(14),
      },
    },
};

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
System:
    OS: macOS 11.5.2
  Binaries:
    Node: 14.17.6 - /usr/local/bin/node
    Yarn: 3.1.1 - /usr/local/bin/yarn
    npm: 7.23.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 99.0.4844.51
    Edge: Not Found
    Firefox: 99
    Safari: 14.1.2

tested in chrome and firefox, both not working

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve