Use of EVP_PKEY_decrypt() should be more secure

This came up in https://github.com/openssl/openssl/pull/15283#discussion_r772433609.
Looks like ERR_clear_error() is not really constant-time due to
static ossl_inline void err_clear(...) in crypto/err/err_local.h
calling OPENSSL_free(es->err_file[i]) and OPENSSL_free(es->err_func[i]), where es->err_file[i] and es->err_func[i] should be NULL if and only if no error queue entry is present at index i.

For timing-sensitive code, one shouldn't be pushing sensitive information to the error queue in the first place. Trying to make the error queue itself constant-time is a losing proposition.
I'm not familiar enough with CRMF to, at a glance, point out what that code should be doing instead, but if you're relying on a constant-time error queue, that means some layer somewhere isn't calling or exposing the right API. Assuming this is some attempt to work around flaws in RSAES-PKCS1-v1_5 (if it's OAEP, this shouldn't be necessary), merely merging failure and wrong size conditions together doesn't seem sufficient. TLS needs to do a constant-time swap to a random secret.

Thanks @davidben for your swift reaction. While its does not answer the question I posted here, it addresses the motivation for the question.

For timing-sensitive code, one shouldn't be pushing sensitive information to the error queue in the first place. Trying to make the error queue itself constant-time is a losing proposition.

Good point. Even the fact that an error entry is pushed is a problem and not even close to constant-time (in comparison to not pushing an error). ERR_raise() usually even allocates file and function name strings on the heap, calling OPENSSL_strdup().
BTW, I wonder why the function
static ossl_inline void err_set_debug(...) defined in err_local.h
that is invoked indirectly from the macro ERR_raise() and has a body of 11 lines is defined as inline,
and likewise various other functions defined there.

I'm not familiar enough with CRMF to, at a glance, point out what that code should be doing instead, but if you're relying on a constant-time error queue, that means some layer somewhere isn't calling or exposing the right API. Assuming this is some attempt to work around flaws in RSAES-PKCS1-v1_5 (if it's OAEP, this shouldn't be necessary), merely merging failure and wrong size conditions together doesn't seem sufficient. TLS needs to do a constant-time swap to a random secret.

Well, I am meanwhile pretty sure that the suggestion by the reviewer at https://github.com/openssl/openssl/pull/9774#discussion_r321224326 and https://github.com/openssl/openssl/pull/9774#discussion_r321311665 to use constant-time failure checking was irrelevant, even misleading.
At that point in OSSL_CRMF_ENCRYPTEDVALUE_get1_encCert() an encrypted newly enrolled certificate has been received and is being decrypted. This only happens under the control of the (CMP) client that had requested a certificate, resorting to a very unusual indirect way of doing proof of possession (while usually the key material related to the cert can be used for signing and thus direct signature-based POP can be done instead).
Pretty sure this decryption cannot be triggered form outside in a tight loop.

Pretty sure this decryption cannot be triggered form outside in a tight loop.

I don't think that's a good criteria for skipping side channel protections. Moreover, the RSAES-PKCS1-v1_5 flaw doesn't even require timing to exploit. Merely being able to probe query ciphertexts for a private key is enough to leak information. Someone who understands both the CRMF protocol and the relevant leaks and attacks here should probably spend more time on this code. (Alas I'm a bit hosed and probably won't have time to figure out enough of CRMF to sort this out. :-( )

I don't think that's a good criteria for skipping side channel protections. Moreover, the RSAES-PKCS1-v1_5 flaw doesn't even require timing to exploit.

I do not know whether this flaw was the motivation to suggest a constant-time result check - @bernd-edlinger can you please clarify what the motivation was?
CRMF is not a protocol, but stands for Certificate Request Message Format,
which is used mostly by the Certificate Management Protocol (CMP).
It is a more flexible but much less known alternative to PKCS#10.

You decrypt something that was received from an untrusted source,
You expect PKCS1 padding and some additional checks which have a certain probability to be satisfied by a randomly
chosen ciphertext, right?
You do not authenticate the ciphertext before it goes to the EVP_PKEY_decrypt function, right?

what is constant time is only this sequence
ERR_raise(no-secret-values-here);
err_clear_last_constant_time(secret-value);
ERR_clear_error();
I think also this would work
ERR_set_mark();
ERR_raise(no-secret-values-here);
err_clear_last_constant_time(secret-value);
ERR_pop_to_mark();

You decrypt something that was received from an untrusted source, You expect PKCS1 padding and some additional checks which have a certain probability to be satisfied by a randomly chosen ciphertext, right? You do not authenticate the ciphertext before it goes to the EVP_PKEY_decrypt function, right?

Thanks for making clear what the concern is, or could be.
Given this information, it is pretty clear that it does not apply,
simply because the encrypted cert/stuff comes from a CMP server that has been authenticated.
(Well, if the client has been misconfigured such that not proper authentication is done then it is not the fault of the CMP+CRMF lib).

Yes, when the message containing the cipher text is authenticated with a signature from a trusted CMP server
you don't need that.  Which signature algorithm is it?

In CMP the message (+ sender) authentication is flexible.
It may be MAC-based with a shared secret (usually, PBM),
otherwise (which is the normal situation) it is signature-based with the algorithm given in the signer cert,
which of course is accepted only if it can be traced to a trust anchor,
which indirectly determines via the CA policies the (minimal) strength of the signature.

So given that the CMP client does not offer an interface for chosen ciphertext attacks with unauthenticated input from an unknown party,
can we conclude that we do not need to take special care in terms of constant-time behavior when checking the results of decrypting parts of an authenticated message?

that's right, and there is no need to clear the error stack in that case either.

Thanks @bernd-edlinger for this clarification.
So we can safely remove from crmf_lib.c the constant-time way of checking the decryption result because at that point it is just misleading,
and we are free to handle the error queue in the way it fits best from other perspectives.

namely whether ERR_clear_error() really is constant-time

in general not.
It is only constant time, immediately after the function
err_clear_last_constant_time and only with respect
of whether this last entry was cleared or not.

Interesting to be pointed by your responses to err_clear_last_constant_time(), which I was not aware of.
BTW, there is no internal documentation for any of the (internal) *constant_time* functions (besides code comments for part of them).
Looking at the implementation of err_clear_last_constant_time() I'd say it is constant-time modulo any still due initialization actions by ossl_err_get_state_int(), which likely do not depend on the error state.
Yet I do not believe that ERR_clear_error() is constant-time,
regardless whether err_clear_last_constant_time() was called before,
simply because (as mentioned above) it frees a couple of strings per error queue entry.

The term constant-time means something different in cryptography.
The point is there is no correlation between the time that ERR_clear_error()
takes and the secret data value that has been passed to err_clear_last_constant_time().
The secret is only revealed if you call ERR_get_error() or similar.

Well, the only data explicitly passed to err_clear_last_constant_time() is a flag whether to clear the last entry, and agreed it is constant-time w.r.t. this input.
Surprising that here the input flag is considered secret, but this makes sense to me because it may express whether a decryption was successful or not.
In the sense I understand your explanation, every function that has no parameter, in particular ERR_clear_error() is trivially constant-time, right?
And this would be the case regardless in which situation it is called, in particular, independent of whether err_clear_last_constant_time() was called immediately before or not, while you wrote earlier that this matters here, which seems contradictory.
So, the original question seems to boil down to this:
Can implicit parameters (i.e., statically accessible values) defeat the constant-time property?

In the sense I understand your explanation, every function that has no parameter, in particular ERR_clear_error() is trivially constant-time, right?

No, secrets may be in globals. Attackers, of course, do not care whether a leakage happened to come from a global variable or not. Indeed the error system is a pile of globals, and part of the issue here seems to be whether there was a secret in the error queue state.
Personally, I think having secret error queue state is too fragile and we should instead not put them in there in the first place. But for RSAES-PKCS1-v1_5, that means different APIs and a change to CRMF's code to adapt to whatever that API is. Indeed, without having looked carefully at CRMF, I worry you may be missing the forest for the trees. The reason for all this timing-sensitive stuff around error handling is because the encryption primitive used by CRMF, RSAES-PKCS1-v1_5, is plain broken. See the Bleichenbacher attack. All the timing sensitive stuff we do around errors is workarounds to mitigate it for certain protocols. https://datatracker.ietf.org/doc/html/rfc5246#section-<IP_ADDRESS> describes what TLS does.
This works in TLS because it effectively combines the RSA padding check with the Finished hash that comes right after it. But it's ultimately a workaround and makes some assumptions around how the key is used. The real answer is to retire the scheme altogether, as TLS 1.3 did when it removed RSA-encryption-based cipher suites altogether.
How all that translates to CRMF, I'm not sure. I'm not familiar with it and don't have the time to dig into it carefully right now.

I do not understand the details of RSAES-PKCS1-v1_5 and honestly do not want to know about them.
As mentioned before, for CMP and CRMF its use is very rare and depends on the type of key material being used.
Nevertheless, for the case it is chosen, the implementation should of course be on the safe side.
Comparing the code in crmf_lib.c with at other calls of EVP_PKEY_decrypt() in crypto/, I found two things:

There usage patters are very similar, see pkcs7/pk7_doit.c, evp/p_open.c, and cms/cms_env.c, e.g.:

    if (EVP_PKEY_decrypt_init(pctx) <= 0)
        goto err;

    if (EVP_PKEY_decrypt(pctx, NULL, &eklen,
                         ri->enc_key->data, ri->enc_key->length) <= 0)
        goto err;

    ek = OPENSSL_malloc(eklen);

    if (ek == NULL) {
        ERR_raise(ERR_LIB_PKCS7, ERR_R_MALLOC_FAILURE);
        goto err;
    }

    if (EVP_PKEY_decrypt(pctx, ek, &eklen,
                         ri->enc_key->data, ri->enc_key->length) <= 0
            || eklen == 0
            || (fixlen != 0 && eklen != fixlen)) {
        ret = 0;
        ERR_raise(ERR_LIB_PKCS7, ERR_R_EVP_LIB);
        goto err;
    }

None of these care to use any constant-time error checking
None of them bother with potentially critical error queue entries.

My conclusion form these observations are:

The common use of EVP_PKEY_decrypt{,_init}() should be factored out in a helper functions used in all such places.
Either those other uses are unsafe w.r.t. RSAES-PKCS1-v1_5 or all the concerns raised about the CRMF implementation were misleading.

evp/p_open.c just assumes that the caller does mitigation against a possible Bleichenbacher Attack.
There is not much that can be done here, One would need to look at callers if they do things like signature checking
before calling this function.
The other two places are different.
There are already some counter measures against Bleichenbacher Attack, or MMA Attack,
(million message attack) which you find in the comments here:
https://github.com/openssl/openssl/blob/2080da84a49b0c52fc8c6e6caef5d373235bd3e4/crypto/pkcs7/pk7_doit.c#L593-L601
It's called MMA  in some comments, because the attacker needs the padding oracle
to decrypt around a million messages in order to forge one signature, or decrypt one
cipher text without ever learning the private key.
So this is even more than a (compromised) CA could do, a CA could easily create a fake
certificate and sign something with this certificate, but it could not decrypt any cipher text.
So yes the other places pkcs7/pk7_doit.c and cms/cms_env.c are unsafe, and they could
be improved.  They are in it's current form since 5840ed0cd1e6487d247efbc1a04136a41d7b3a37
where I improved the countermeasures to require 100 times more messages than previously
by additionally checking the expected length of the plain text.
Well I have not yet made the function EVP_PKEY_decrypt constant-time at that time.
So today they could be formally constant time, but that should be done for
the whole process.
And if the condition fails the a deterministic default- key should be used
instead of a random one.  That should be based on something that the
attacker can't possibly know, for instance SHA256(CIPHER_TEXT|PRIVATE_KEY).
so it is not possible that the attacker can see a different behaviour
when she sends the same CIPHER_TEXT again, yet she can't predict what
exactly will be decoded.  The cms part shoud also use a default key
at least when the RSAES-PKCS1-v1_5 padding is used.

Thanks a lot @bernd-edlinger for your clarifying further response.
According to the evolution of this discussion,
I changed the title of this issue and promoted it to a (minor?) security issue/bug.

How about factoring out from evp/p_open.c, crmf/crmf_lib.c, cms/cms_env.c, and pkcs7/pk7_doit.c the code sections calling EVP_PKEY_decrypt{,_init}() (and possibly also the EVP_PKEY_CTX handling) into a common helper function?

How to proceed here?

@bernd-edlinger, can you provide a concrete solution here, ideally a PR?

This has been open since 1.5 years.
@bernd-edlinger, how do you suggest to proceed?

How about factoring out from evp/p_open.c, crmf/crmf_lib.c, cms/cms_env.c, and pkcs7/pk7_doit.c the code sections calling EVP_PKEY_decrypt{,_init}() (and possibly also the EVP_PKEY_CTX handling) into a common helper function?

This is meanwhile done via #17354.
So any improvements in this topic can now be done at one place,
namely in evp_pkey_decrypt_alloc() in crypto/evp/asymcipher.c.


Template bundling

Template bundling doesn't yet work, so the tool expects to find templates in a relative path. They should get installed somewhere when the tool is installed but that's an element of Rust packaging I've not yet used.

Might need to use https://doc.rust-lang.org/std/macro.include_bytes.html


ModuleFederationPlugin ignores singleton: true when requiredVersion is false

Bug report

What is the current behavior?
When creating a new ModuleFederationPlugin with shared libraries, if one of the packages has a configuration of singleton: true, requiredVersion: false, it will produce a remote entry file that uses the "loadFallback" function to try and get the package. This can lead to package duplication if a newer version of the library is provided by this module (or another module).
If the current behavior is a bug, please provide the steps to reproduce.
Sample repo: https://github.com/colacadstink/WebpackModuleFederationBug
The dist directory is created from the standard webpack.config.js, and the workingDist directory from the webpack.working.config.js file. The only differences between these files are the output directories and the requiredVersion of lodash.
Note the differences between dist/remoteEntry.js and workingDist/remoteEntry.js, specifically on line 380:

The dist copy of the file is using the loadFallback function, which will always return the highest version number available, regardless of if another instance has already been created; this can lead to situations where more than one instance is constructed, breaking the singleton flag.
The workingDist copy uses the loadSingletonVersionCheckFallback function with a version argument of [0]. This will let us use whatever version of the package is currently loaded, regardless of version, which is the same behavior I'd expected with the requiredVersion being false.

What is the expected behavior?
I feel as if this should be acting similarly to what happens if the requiredVersion is set to '*', which causes the method used in the remote entry file to be "loadStrictSingletonVersionCheckFallback" with a version argument of [0] (which does work as expected).

Other relevant information:
webpack version:  v5.64.2
Node.js version: Tested on v12.16.1 and v12.20.1
Operating System: n/a, but definitely there on Windows
Additional tools: I used Lodash for demonstration purposes, but there's nothing special about it.

Minor clarification: Using "" is not identical to using false. false should (again, as I understand it) accept all versions, including pre-release versions like 1.0.0-SNAPSHOT, whereas "" will reject that version (as it should per semver standards). So right now, if I want to have a singleton instance of a package, not care what version it is, including allowing pre-release versions, I cannot do so without potentially having two instances of my package running around at the same time.


instruction updates to improve usability

Updated README and instructions returned by the app itself to use querystring parameters instead of separate arguments for curl command.

@trieloff I had an idea this would be the goal.  There is therefore nothing to change for documentation, and this PR will be closed.
We will need a different PR though, the implementation needs to be fixed because the POSTed fields do not seem to work.
% curl -X POST -F "domain=adobe.com" https://eynvwoxb7l.execute-api.us-east-1.amazonaws.com/helix-services/domainkey-provider/v1/
No domain specified

https://github.com/adobe/franklin-domainkey-provider/blob/main/src/index.js#L38-L39 is apparently finding domain to be empty.  I will spend some time troubleshooting today, may come back if I need help.

It appears context.invocation.event.body has a Base64-encoded value showing the POSTed field.  I can probably make the change here, but I wonder if this is better solved in a more global utility-type repo?  Please advise.

Closing PR and deleting branch, will reopen with fewer commits.


Modded Dimension support

Hello, it's me again!
This time, I would like to suggest to support generating snow under trees in other dimensions mod, for example in the Blue Skies dimensions, because it looks like this :

Looking into this, I was able to verify that Snow Under Trees does indeed add its feature to the snow covered pines biome (the one in the screenshot) of this mod. From that side, it should work in this modded dimension, because as long as the feature is added to a biome, it should work no matter the dimension (vanilla, datapack, modded, ...). However, checking whether the feature is still added to this specific biome when it's being generated yields a no (contrary to other biomes where Snow Under Trees does work). As far as I know, this can really only happen if something resets the generation settings of the biome at some point between Snow Under Trees adding its feature to them and Minecraft generating the world. Thus I conclude this is a bug out of my control.


Middle click on a tab or closing it should not focus it

Either a middle click on a tab of closing it via the x is a strong indicator that the user wants to close the tab. I think we should act more like a browser here and not open it beforehand as it's extra work laying loading the file, initializing/laying out the editor, etc.

Maybe, not sure though. The reason the editor comes to the front is to help the user make that decision the dialog is asking for so that you can review the changes before in the end discarding all your changes. I find that more useful than having to pay the extra price of loading the editor.

Makes sense, let's close


Add Preliminary PeriodicAdd Executable (No Rename or Delete)

Looks good mod comments.  Eventually we'll want a separate daemon that crawls the client directories, rather than building it into the program that does the search.  But this is ok for getting started.

For factoring out the grep command, I've added some funky test code. So maybe a quick look at client_test.go?

Finished a pass, looks fine other than the regression bug with the last index update file.
Tests look fine. Ideally there would be one that actually checks a case that returns different results for search and searchStrict.  If that's hard to do though, it's fine how it is.


Custom menus follow up

Some follow up from code review that I consider more debt and not bugs which is fine to look at for July:

[x]  remove the titlebarPart.css transition rule unless there is a need for it
Looks like it is being used (https://github.com/Microsoft/vscode/pull/53211#issuecomment-401033547)
[ ] remove the old menu implementation

[ ] verify everything works as before when menu is not custom and on macOS

[x] remove the vscode.toggleAutoSave which I think is no longer needed then
[x] remove FILE_MENU_FAKE_OPEN_FILE_COMMAND_ID ?
[ ]  why are there two AlternativeKeyEmitter (here, here), can they not be reused/shared?
[ ] what is the purpose of the resizer element in the title bar part?
[ ] why are the window buttons called chrome-xy? google chrome?
[x] do not use a querySelect to find the element, rather keep it as field
[x] menubarPart is should use the builder pattern either fully or not, but partially is a bit weird, e.g. this.customMenus[menuIndex].titleElement.getHTMLElement().onmouseenter vs. this.customMenus[menuIndex].titleElement.on(EventType.CLICK, (event) => {
[ ] menubarPart is not properly cleaning up disposeables, for example this:

this.configurationService.onDidChangeConfiguration(e => this.onConfigurationUpdated(e));
should be
this._register(this.configurationService.onDidChangeConfiguration(e => this.onConfigurationUpdated(e)));

@bpasero

why are there two AlternativeKeyEmitter (here, here), can they not be reused/shared
I originally created the second one because the original was used for the alternative actions throughout the workbench, but that behavior was still under discussion. Now, I've modified it by quite a bit to fit my needs and changed the name even.

what is the purpose of the resizer element in the title bar part?
On windows, the window can't be resized from a draggable region, so the resizer let's you grab from the top.

why are the window buttons called chrome-xy? google chrome?
UI jargon, that stuffs called window chrome.


Added My NewsApp Project.

This project is to view the latest news in the respective category with the content being refreshed and news fetched after every hour. Made with the help of NewsApi.
Here are the screenshots for the same.

Please review the code and merge the PR. Thank you for the support.

Hey! I changed the directory as per the requirements. Please see.


Failed loading lang files

3.4.4 buid 511
i tried remove all files .. reload, restart server .. nothing please help
[23:33:52 ERROR]: [Quests] Failed loading lang files for cs-CZ because they were
not found. Using default en-US

Duplicate of #481
Try the latest dev build and check your folder permissions.
http://ci.ac3-servers.eu/job/Quests/510/

If you're on Windows 10, right-click on your /plugins folder, select Properties, and go to the Security tab. Click "Advanced" and then, in the Permissions tab, select "Change Permissions". Choose "Add" and in the Select User or Group window, type "everyone" in the text box. Click "Check Names" and then choose "OK". Continue clicking "OK" until all windows are closed.

Now, backup your Quests folder and delete the original (you can copy over your data from the backup later). Try running the server, allowing Quests to generate new files, and see if the issue is resolved.

solved thanks


This is not reading my 2nd  tags, it says I still need to add them.

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.98 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
  I tried a hard refresh, I tried cutting some of the text out so it was not exactly like the first paragraph, I tried switching the paragraphs, nothing worked.

@stutzro As the instruction says -

Create a second p element after the existing p element with the following kitty ipsum text: Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

The content of your second p element is wrong. Please visit the Help Room if you need help with this challenge.
Happy Coding!

Thanks @stutzro for reporting, please proceed as suggested by @Manish-Giri !
@Manish-Giri thanks for commenting.

But the instructions don’t say this, they say to just use some of the content already there.  I tried altering the text as well and it did not work.
Thanks,
Ronnett Stutz
UVU Student Affairs
Web Developer
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Office: LC 228
From: Manish Giri <notifications<EMAIL_ADDRESS>Reply-To: FreeCodeCamp/FreeCodeCamp <reply<EMAIL_ADDRESS>Date: Wednesday, December 21, 2016 at 2:40 PM
To: FreeCodeCamp/FreeCodeCamp <FreeCodeCamp<EMAIL_ADDRESS>Cc: UVU <stutzro<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [FreeCodeCamp/FreeCodeCamp] This is not reading my 2nd  tags, it says I still need to add them. (#12157)
@stutzrohttps://github.com/stutzro As the instruction says -
Create a second p element after the existing p element with the following kitty ipsum text: Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.
The content of your second p element is wrong. Please visit the Help Roomhttps://gitter.im/FreeCodeCamp/Help if you need help with this challenge.
Happy Coding!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/FreeCodeCamp/FreeCodeCamp/issues/12157#issuecomment-268647268, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXivACpSvUt6xcSStwuUrQrk0ZIgRHdkks5rKZy4gaJpZM4LTVyM.


add animation for data lines and points

Displayed points for series in the graph (commit from pryaniq, see https://github.com/pryaniq/SwiftChart/commit/5550bf1f79bf8f8827f96b941194ccad2da7270a)
Animate line drawing and point drawing. This can be configured via the new Struct ChartAnimation.
Howto:
chart.animation.enabled = true/false
chart.animation.duration = 1

Animation is disabled by default. Also updated the sample project.

Hi @dhf thanks a lot for you PR!
The idea of supporting animations is tracked here (see also this PR) but the reason it's not implemented yet is that it is more difficult to implement as I thought.
Look for example how the animation works with the others examples:

With a single line chart it seems performing well, but

the animation's duration depends on the number of values, e.g. if there are less values it seems way too slow.
the lines above and below the x-axis are rendered separately (so we can give a different color), hence the animation looks weird
background of filled series is not animated

I think we should find another way, maybe using a mask over the chart. I even would prefer to have the dots in a different PR, since it is another kind of enhancement. @pryaniq PR needs to be reviewed a bit, see discussion.


disableFocus and onAudioFocusChanged don't work

Bug

Environment info

React native info output:
  React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (8) x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
      Memory: 45.70 MB / 16.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 8.11.3 - ~/.nvm/versions/node/v8.11.3/bin/node
      Yarn: 1.13.0 - /usr/local/bin/yarn
      npm: 5.6.0 - ~/.nvm/versions/node/v8.11.3/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
      Android SDK:
        API Levels: 23, 25, 26, 27, 28
        Build Tools: 23.0.1, 25.0.2, 27.0.3, 28.0.2, 28.0.3
        System Images: android-27 | Intel x86 Atom, android-27 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>99772
      Xcode: 10.2.1/10E1001 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.10 => 0.59.9

Library version: 4.4.2
Steps To Reproduce

Add

onAudioFocusChanged = (event) => {
    this.setState({ paused: !event.hasAudioFocus })
    this.props.onAudioFocusChanged(event)
  }

onAudioFocusChanged={this.onAudioFocusChanged}

or disableFocus={true}
2. Make a call to the phone on which the app is working
Describe what you expected to happen:

Playback stops when call happens

It works in debug, but not in release APK.

This props are not documented AFAIK, but will take a look.
Is that on Android or iOS?

Please test on this PR, it is still not released: https://github.com/react-native-community/react-native-video/tree/release/4.4.3

@CHaNGeTe I tried it, still didn't help. But thanks

I discovered that when I plug my headphones, make a call to my phone, onaudiofocuschange gets called and everything works. But if i don't use headphones and listen through speakers instead, onaudiofocuschange doesn't get called. It's an important issue.


Update PPP, fix DG typos and formatting

Codecov Report

Merging #251 (f867304) into master (3d53384) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #251   +/-   ##
=========================================
  Coverage     65.17%   65.17%
  Complexity      850      850
=========================================
  Files           137      137
  Lines          3136     3136
  Branches        414      414
=========================================
  Hits           2044     2044
  Misses          951      951
  Partials        141      141

Continue to review full report at Codecov.

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


Add Stack Exchange

Ref. file from https://stackoverflow.com/company/logos
Background color from the https://stackexchange.com/ title bar (.top-bar)
Minified using https://jakearchibald.github.io/svgomg/

This is really good! Please can you make a few changes.

Remove the class="...." stuff. It isn't needed.
You can make it smaller by rounding up the decimal places.
Please follow https://github.com/edent/SuperTinyIcons/blob/master/CONTRIBUTING.md - and then I'll merge it.

Thank you once again :-)

Class removed, added to ref & radius.
I don't have/know the tools to round the decimals automatically. As this is not the final size, I did not add it to readme.md.


TC Cache Config RPM does not depend on git and perl unless built on RHEL 6, 7, or 8

I'm submitting a ...

bug report

Traffic Control components affected ...

Traffic Control Cache Config (t3c) RPM spec file

Current behavior:

Unless the RPM build distribution is RHEL 6, RHEL 7, or RHEL 8, an RPM from trafficcontrol-cache-config.spec will have no dependencies.
Expected behavior:

trafficcontrol-cache-config should depend on git and perl:
https://github.com/apache/trafficcontrol/blob/f126954b4f8753ffa2be9006f996a2a971484a9a/cache-config/build/trafficcontrol-cache-config.spec#L29-L31
Minimal reproduction of the problem with instructions:

Go through the steps to build the ATC RPMs natively

Build the TC Cache Config RPM outside of Docker, assuming you are not on CentOS 6, CentOS 7, or CentOS 8:
cd infrastructure/cdn-in-a-box
make native cache/trafficcontrol-cache-config.rpm
rpm -q --requires cache/trafficcontrol-cache-config.rpm

Output of rpm -q --requires cache/trafficcontrol-cache-config.rpm that reproduces the bug:
/bin/sh
/bin/sh
/usr/bin/perl
config(trafficcontrol-cache-config) = 6.0.0-11401.60a9ffdc.el8
perl(Getopt::Long)
perl(strict)
perl(warnings)
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(FileDigests) <= 4.6.0-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rpmlib(PayloadIsXz) <= 5.2-1

based on the reproduction instructions' output, it does look like the RPM depends on a Perl interpreter at /usr/bin/perl as well as packages which themselves depend on Perl e.g. perl(strict) - do those really not result in an implicit dependency on Perl?

That's just a result of rpmbuild detecting a script with a Perl shebang. We should still explicitly require the dependency until the Perl script is removed.


[help wanted]

https://github.com/codelv/enaml-native-icons/blob/7b22df746899b84c278cc36939ff01d0e4284b67/src/iconify/cryptocoins.py#L3
how do you extract  u'\uE001' from ttf fonts

explained in gitter Lobby


difficulty with conjugacy with dcat dependents

We need a dcat-ddirch conjugacy for DPs. However, as in other cases, dcat is tricky because it is a scalar distribution with a multivariate node. This causes problems with generation of the conjugate sampler and there are multiple directions we could take to try to address this. Will try to lay out what I see as the possibilities here.
Here's the line for the conjugacy definition:
dcat      = list(param = 'prob', contribution_alpha = 'calc_dcatConjugacyContribution(length(prob), value)')),

that would be added to the ddirch conjugacy list.
Note that in the discussion below, it will be relevant that we only need the prob parameter to know its length; we don't need the value of prob, so part of one of the solutions below is to set up dcat so that it has a "parameter" that is d=length(prob) and then we would have:
dcat      = list(param = 'prob', contribution_alpha = 'calc_dcatConjugacyContribution(d, value)')),

The basic problem is that when I try to use the first form of conjugacy above, the setup code ends up trying to use a variable dep_dcat_nodeSizeMax that doesn't exist (because earlier in the generation of the setup code we check and find that dcat is a scalar density.
So this code causes problems:
dep_dcat_prob <- array(0, dim = c(N_dep_dcat, dep_dcat_nodeSizeMax))

In fact as mentioned above, and related to issue #557, we don't really need the values in dep_dcat_prob to do our conjugacy calculations. We only need it for length(prob).
So I see two possible avenues:

Modify our conjugacy handling so that we correctly deal with figuring out the size of dep_dcat_prob.

In combination with issue #557, omit dep_ddist_dparam objects that are not needed for conjugacy calculations and do some processing of dcat declarations to insert a d=length(prob) parameter that we can then use in the dcat-ddirch conjugacy. In this case, the dimensionality issue should go away because both value and d for dcat are scalars.

There might be other options. @danielturek please see what you think and let's chat about how to proceed. Not particularly urgent but would be nice to address for 0.6-7 when we will hopefully have initial DP models in nimble.
The code is in dp_conjugacies and here is an example model:
   set.seed(0)
    n <- 100
    alpha <- c(10, 30, 15, 60, 1)
    K <- length(alpha)
    p <- c(.12, .24, .09, .54, .01)
    y <- rmulti(1, n, p)
    y <- rep(seq_along(y), times = y)

    code <- nimbleCode( {
        for(i in 1:n)
          y[i] ~ dcat(p[1:K])
        p[1:K] ~ ddirch(alpha[1:K])
        for(i in 1:K) {
            alpha[i] ~ dgamma(.001, .001);
        }
    })
m = nimbleModel(code, data = list(y=y), constants = list(n=n, K=K))
conf = configureMCMC(m)

@paciorek  Fixed, changed pushed to branch dp_conjugacies.  Tested through successful compilation, and running the compiled MCMC.
Note, I did not test the correctness of the posterior of the ddirch-dcat conjugacy.  Only that it executes and samples without errors.  The correctness of the conjugate sampling should still be checked.
I'm closing this issue for now.  If you find there's still a problem, please re-open this, or a new issue.


ssh multiplexing?

Can this be used to handle ssh over TLS with SNI routing?

To better understand, do you mean the ssh connection is wrapped in TLS to hide the SSH bytes (like what's described here)? Or to serve 2 services on the same port and respond based on the peeked bytes (as in #17)?
If it's the latter, then you can do that by combining the layer4 app with the SSH app, as Matt said on that other issue. If it's the former, then not in its current shape, but I'm intrigued and will consider it a feature request.

You could probably do the former with layer4 too. Just terminate TLS and proxy the bytes to the ssh server.

FYI: I'd recommend OpenSSL s_client or Telebit sclient over socat - just due to the sheer complexity of the docs of socat vs the simplicity of openssl s_client and sclient (which I literally created exactly for this use case, but on the individual scale, and to work on Windows clients).

Thanks for that sample config. That makes sense at a glance. I'll try it out tonight.

This doesn't really have to do with caddy-ssh (now kadeessh as of a few hours ago?) at this point, but since this is where search has led me and where the conversation has evolved here, I want to provide the solution here as well.
Full Instructions

caddy online build is currently broken due to go bug, must use xcaddy
must build caddy with caddy-l4 and its plugins
sample config for terminating tls on a full port capture (no sni, no multiplexing)

1. Install xcaddy
Download xcaddy, make it executable, and put it in your PATH (pathman can help if you're unfamiliar with that):
pushd /tmp/
curl -o ./xcaddy_0.3.2_linux_amd64.tar.gz \
    -L https://github.com/caddyserver/xcaddy/releases/download/v0.3.2/xcaddy_0.3.2_linux_amd64.tar.gz
tar xvf ./xcaddy_0.3.2_linux_amd64.tar.gz
chmod a+x caddy
mv ./caddy ~/bin/caddy
popd

2. Build with layer4
Needs layer4, l4tls, l4ssh, l4proxy. Using duckdns as an example
#!/bin/sh

#export XCADDY_SETCAP=1
export XCADDY_SUDO=0
export XCADDY_SKIP_CLEANUP=1

xcaddy build \
    --with github.com/mholt/caddy-l4/layer4 \
    --with github.com/mholt/caddy-l4/modules/l4tls \
    --with github.com/mholt/caddy-l4/modules/l4ssh \
    --with github.com/mholt/caddy-l4/modules/l4proxy \
    --with github.com/caddy-dns/duckdns

3. Test 1 TLS to SSH

my-sshtls is an arbitrary name
<IP_ADDRESS> is the IP of my server
<IP_ADDRESS>:22443 means that the server is listening for TLS-SSH on any network interface on port 22443 (because it consumes the entire port - no multiplexing yet)
localhost:22 is for the network connection, not indicative of hostname or SNI, etc

{
  "logging": {
    "logs": {
      "default": {
        "level": "DEBUG"
      }
    }
  },
  "apps": {
    "layer4": {
      "servers": {
        "my-sshtls": {
          "listen": ["<IP_ADDRESS>:22443"],
          "routes": [
            {
              "handle": [
                {
                  "handler": "tls"
                },
                {
                  "handler": "proxy",
                  "upstreams": [{ "dial": ["localhost:22"] }]
                }
              ]
            }
          ]
        }
      }
    },
    "tls": {
      "certificates": {
        "automate": ["localhost", "<IP_ADDRESS>"]
      },
      "automation": {
        "policies": [
          {
            "issuers": [
              {
                "module": "internal"
              }
            ]
          }
        ]
      }
    }
  }
}

4. SNI Routing to SSH
Working this out now.
5. Multiplexing HTTPS and SSH
Will work this out next

I guess this is still a WIP ? Would like to try this out but it’s not merged yet into this repo ?

I guess this is still a WIP ? Would like to try this out but it’s not merged yet into this repo ?

There isn't any necessary work here nor are there any necessary PRs to merge. You can use TLS-terminated SSH sessions as-is using a combination of caddy-l4 and kadeessh. It's a matter of configuration, which I could include as an example on the README or the wiki. There's a working config sample by me above. I kept it open to remind myself to include the example config of this use-case somewhere handy for users.
Anything specific you're missing?


docs: contributing pages

### Tasks
- [ ] create a page for [GitHub workflows](https://docs.argilla.io/en/latest/community/contributing.html). We can copy-paste this reasoning and page.
- [ ] create a page for the [Development environments](https://docs.argilla.io/en/latest/community/developer_docs.html). Feel free to adapt this to the current deployment of Argilla UI, docs (mkdocs now) and server (@jfcalvo), and SDK 2 (@burtenshaw )

@davidberenstein1957 What's the status of this issue? Is this something still pending to do? If so, is it for 2.0 or can be left for later?

@nataliaElv We have basic outline of these pages, but let's move it to 2.1
@davidberenstein1957 Should we expand on this issue to improve what we have?


Doesn't hot reload rails anymore

overmind doesn't reload the assets server from Procfile.
I don't know how to trace/debug it.
but only rails server starts
I tried hivemind and it seems working fine with it.
So not sure what would be the issue and how to debug it.
I used the eveilchat repo from evilmartians

Turned out it can't read Procfile.dev only Procfile


Bug 2106666: Return error when VM's info returned from VC doesn't contain IP address

This backports a fix for the Cloud provider node manager that fixes an issue where, after initialising correctly with IP addresses, the CCM will, after a few minutes, remove the IP addresses from the nodes.
I've observed this behaviour manually and believe, based on the descriptions of the upstream PR and issue (details in BZ) that this should resolve the issue. We aren't seeing the issue in 4.11 so I've based the BZ chain on the rebase which included this commit.

/approve

/label backport-risk-assessed

/label cherry-pick-approved


Use multiple attempted delivery codes for USPS tracking

About
There are multiple possible attempted_delivery codes - see page 335 of this, or look at this screenshot:

There are multiple codes for "Notice Left"/Attempted Delivery - rather than just use code 55, use all of the ones in the appendix. Code H0 came from me finding responses manually through a database of tracking codes, seems to also be Attempted Delivery (unsurprisingly not documented).
Changes
Add an array of known ATTEMPTED_DELIVERY_CODES and check for inclusion when saving the attempted_delivery_date.
This fixes part of the red build, which doesn't actually depend on this fix - it was just an outdated tracking number. The other half is fixed by https://github.com/Shopify/active_shipping/pull/362
Review: @kmcphillips @RichardBlair @mdking @MalazAlamir

🐑 :it:

🚢

Nice find! 👍


Document a couple of API example scripts

closes #250

It should be noted there is a requirement that jq be installed.


Request KolibriOS support

https://en.wikipedia.org/wiki/KolibriOS

There is no Chromium for KolibriOS, but there is a SSH client, so you should be able to connect to remote Carbonyl instance.

Closing this as not supported by Chromium, but feel free to open a PR if you can get a port working


Make the media controller optional

Hello, ich have the issue, that on my Galaxy Watch (which comes with a already built in media controller), I cant get rid of the media controller, which is detected as an extra app, that I cant uninstall, without removing the whole app. Is there any way to get rid of it, because it really bothers me and the icon is very low quality too. Srry for my bad english

There is an option for this in settings


NXP: FlexTimer: Various clean-ups

Various clean-ups for the NXP Kinetis FlexTimer (FTM) PWM driver:

The driver and in-tree board pinmux configurations were changed from using a mix of CONFIG_PWM_* and DT_FTM_* fixups to relying on the DT_INST_* defines.
The FTM instances of the KE1xF SoC was renamed from FTM to PWM* to match the rest of the in-tree SoCs.

While dealing with this issue, I pondered if a better long-term solution would be to add "flags" for specifying the PWM polarity to the pwms device tree phandles (similar to GPIO flags and similar to how Linux handles PWM polarity). I will submit a separate RFC PR for that change.

While dealing with this issue, I pondered if a better long-term solution would be to add "flags" for specifying the PWM polarity to the pwms device tree phandles (similar to GPIO flags and similar to how Linux handles PWM polarity). I will submit a separate RFC PR (#20657) for that change.

Given the positive feedback for #20657 on the API meetings, I have implemented the FTM PWM signal polarity support on that PR. I have kept this PR for the various clean-ups to the FTM driver.


LG TV UI

Missing on theater screen

Ui implemented, but buttons don't send mqtt messages.
The URLs for images in launch points do not fetch from the TV.

Implemented for Tablet.

Implemented for Tablet.


Ability to style toast.promise.

Currently, I am using a toast.promise,  and it would be nice if we had the option to style it, like the others (toast.success, etc).
I apologise in advance if there is an issue that has already been made about this.

Hi! There is a similar issue, where I proposed solution to this problem. Let me know if this works for you.
https://github.com/timolins/react-hot-toast/issues/67#issuecomment-983611798

Hi! Thank you for your quick response. I have added toastOptions to my Toaster component and it worked!
Thank you for your help.


ArgumentOutOfRangeException exception version 3.1.0

I noticed a similar issue in case #70 , which has since been resolved. However, I'm still encountering ArgumentOutOfRangeException with version 3.1.0 sporadically when collections are cleared and re-added. I'll try to work on isolating the problem
System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index')
   at ObservableCollections.FiltableSynchronizedViewList`2.get_Item(Int32 index)
   at ObservableCollections.NotifyCollectionChangedSynchronizedViewList`2.System.Collections.IList.get_Item(Int32 index)
   at DevExpress.Xpf.Editors.Helpers.PlainDataView.ProcessAddItem(Int32 index)
   at DevExpress.Xpf.Editors.Helpers.DataView.ProcessChangeSource(ListChangedEventArgs e)
   at DevExpress.Xpf.Editors.Helpers.DataController.ItemsSourceListChanged(Object sender, ListChangedEventArgs e)
   at DevExpress.Data.Utils.WeakEventHandler`3.OnEvent(Object source, TEventArgs eventArgs)
   at DevExpress.Data.Helpers.BindingListAdapterBase.OnSourceCollectionChanged(Object sender, NotifyCollectionChangedEventArgs e)
   at DevExpress.Data.Utils.WeakEventHandler`3.OnEvent(Object source, TEventArgs eventArgs)
   at ObservableCollections.NotifyCollectionChangedSynchronizedViewList`2.RaiseChangedEvent(NotifyCollectionChangedEventArgs e)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
   at System.Windows.Threading.DispatcherOperation.InvokeImpl()
   at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(Object obj)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)

Thank you, I'll check it out.
It seems to be going through binding, but could you give me a simple Add/Remove process?

Thank you for your response. I was unable to reproduce this issue with the latest version 3.2.0. I’ll go ahead and close this issue for now

I can now reproduce the problem. It appears to be specific to WPF and is caused by posting values from different threads. I'm attaching the project that demonstrates this issue. Thank you for looking into it!
obstest.zip


Site not building with a submodule referenced in _config.yaml file

I have a submodule (resources/) as part of my Jekyll repo that is referenced in the _config.yaml file:
layouts_dir: /resources/_layouts
includes_dir: /resources/_includes
sass:
  sass_dir: /resources/_sass
  style: compressed

Unfortunately when the jekyll-deploy-action runs, it does not seem to be building with the submodule. The rendered site is not getting the styling that I had been getting when building off of the master branch with the submodule. It should look a lot more like the rest of the site, which is styled using the sass files in resources/.
Any ideas what I might be missing to get the site to build using resources/?

Of course I managed to figure it out after posting the issue - sorry! I modified the actions/checkout step of the workflow to add submodules: true like below...
...
    steps:
      - uses: actions/checkout@v4
        with:
          submodules: true
...


[Question] What's the difference between this tool and owasp-dep-scan

there was another tool: owasp-dep-scan
this caused me confusion, can someone clarify this.

The two are unrelated, independent projects that have both listed their project under the OWASP umbrella. There might be more similar tools in the full list of projects that have registered with OWASP.


Update ClassTransformer.java

After much consideration I have decided this is a non-issue and will not be accepted, sorry.


Add iBook and PowerBook Batteries

I know A1008 is an iBook battery, but don't know which iBook series.

A1061 is another

A1012 as well


Problems with FF for RosettaFold generated proteins

In the post-docking MD pipeline (e.g., small-scale docking), I am having problems running docking and then MD simulations for proteins folded using RosettaFold All-atom.
The docking runs just fine, but the job fails at the moment of preparation of the MD simulation, with the message:
Did not recognize residue SER; did you forget to call .add_molecules() to add it?
If a previous protein-preparation with Maestro is performed on the folded structure, the MD runs normally.
May be solved by adding an extra protein-prep step with PDBFixer.

Can you send the pdb file that is failing?

Here's the pdb file:
mers_apo_protein_rf.pdb.gz
I think the problem is with the protein not being "capped" by the right residue, but I don't know how to fix it with PDBFixer.
I tried the following:
fixer = PDBFixer(str(pdb_file))

fixer.findMissingResidues()
fixer.findNonstandardResidues()
fixer.replaceNonstandardResidues()
fixer.removeHeterogens(True)
fixer.findMissingAtoms()
fixer.addMissingAtoms()
fixer.addMissingHydrogens(7.4)


Determination of the select queries in BaseEngineSpec

Hello!
We are now developing a new SQLAlchemy dialect for Kusto (Azure Data Explorer) and want add support for Kusto database in Superset in nearly future. (This also covers https://github.com/apache/superset/issues/10646.)
Kusto language has two dialects: SQL and KQL. While there is no problem with SQL dialect, we also want to integrate KQL in Superset.
Problem
KQL select queries do not contain the word "SELECT" so they look like
MyTable | take 10 (this is equivalent of Select * from MyTable Limit 10).
The absence of the "SELECT" keyword breaks the limit behavior in SQLLab because it relies on the sqlparse package and we can't add a limit to queries from UI.
Proposed solution
The solution is to make the determination of the select queries in BaseEngineSpec so that you can override it in your own spec. This is the same way how base dialect determines read-only queries with is_readonly_query.
We will provide PR to implement this small feature.
Thank you!

Closed by https://github.com/apache/superset/pull/15013


Concurrency exception thrown

I'm debugging an app using the -com.apple.CoreData.ConcurrencyDebug 1 option. During that debugging session I found that RestKit 0.24.1 crashes on the line id relationshipValue = [object valueForKeyPath:destinationKeyPath] of the method RKManagedObjectsFromObjectWithMappingInfo inside the RKManagedObjectRequestOperation.m. It should be line# 380.
The code gets called by NSSet *managedObjectsInMappingResult = RKManagedObjectsFromMappingResultWithMappingInfo(mappingResult, self.mappingInfo) ?: [NSSet set]; inside deleteLocalObjectsMissingFromMappingResult:
I don't see a crash, if I safeguard the call with a performBlockAndWaitlike the following snipped
// Proceed with cleanup
__block NSSet *managedObjectsInMappingResult;
[self.privateContext performBlockAndWait:^{
    managedObjectsInMappingResult = RKManagedObjectsFromMappingResultWithMappingInfo(mappingResult, self.mappingInfo) ?: [NSSet set];
}];

Id love a pull request with the necessary fix!

https://github.com/GrahamDennis/GDCoreDataConcurrencyDebugging
Useful framework here. I see a lot of issues being exposed with it. I think related to a lot of the crashes I'm seeing.

GDCoreDataConcurrencyDebugging is a nice option, but got dispensable with the new -com.apple.CoreData.ConcurrencyDebug 1launch argument introduced with iOS8. More info here: http://oleb.net/blog/2014/06/core-data-concurrency-debugging/
I did a pull request for this issue, but for some reason the Travis build failed. I can't imagine that such a subtle change can break the build and there was no obvious test that failed.

Interesting. Well at least with the library I was getting warnings all over the place, didn't try with this yet. Looks like your pull just addresses the one spot. Did you see others?

No I only saw this one spot. No other issues popped up with -com.apple.CoreData.ConcurrencyDebug 1enabled. But I never tried the GDCoreDataConcurrencyDebugging library. Could it be that this library creates some "False Positives" as well?


[T2A5] [W09 - Unknown] William Hawken

@Torasian pls update PR name with your team ID

Hi sir,
We haven't resided out team number in our tutorial yet, so should I update when I find out?
Kind regards,
William Hawken
On 24 Aug 2016 9:42 p.m., "Damith C. Rajapakse"<EMAIL_ADDRESS>wrote:
@Torasianhttps://github.com/Torasian pls update PR name with your team ID
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/nus-cs2103-AY1617S1/addressbook-level1/pull/234#issuecomment-242067802, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJEowIGYs14NJGjrJkOQS3OMyRBPGCcEks5qjEovgaJpZM4JsAkW.

@Torasian Your PR id has too many spaced :-P
[T2A5][T13-A2]William Hawken would be the right one!

@Torasian

in general good work on this. keep it up!
ensure next time you add a new function, you need to add tests as well!

btw, please use your full name as it appears in IVLE in the PR titles


Feature Request: Have relation field with "users" table mapping can be auto-filled during time of creation with jwt token.

I have table which has comments, this is the table structure.

here, I have owner as the relation to the default "users" table

If there is a toggle called "autofill" (only for this special "users" table relation), which I can toggle to get the user data auto-populated from the JWT Token of the user, any update happening on this field will have the owner details who updated it.

Sorry, for now there are no plans for this as the request could be performed by entirely different auth collection or even not an auth record (eg. admin or guest).


[rest] Add websocket event server

Currently you can provide a webhookUrl to the AFJ rest agent that we will send webhook events to whenever an event is emitted on the agent. Listening to webhook events means you need to set up your own server that can listen to those webhooks. In some cases it may be desired to get the events from the rest api through a websocket interface. This means you don't need to host a server and can connect directly through a websocket client.
We can reuse the same server that is used for the REST api for the websocket events. This means no extra servers need to be spawned. You can see in the mediator.ts from AFJ how the upgrade to websocket works: https://github.com/hyperledger/aries-framework-javascript/blob/main/samples/mediator.ts#L89. This can be added in the server.ts file.
The websocket handling can be implemented using the ws package: https://npmjs.com/package/ws. We can use this to listen to incoming websocket connections, and we can emit..

Create an EventHandler class that exposes a method to send events. The EventHandler should keep track of the websocket connections, and the webhook url.
Register the websocket server in the server.ts. Whenever an upgrade is made (See mediator.ts example) we should register the connection on the event handler. Whenever the ocnnection is closed, we should unregister the connection on the event handler.
Whenever the event handler is called to send an event it will send the event to the webhookUrl (if registered) and all connected websocket clients.
Document how the websocket server can be used in the rest README file.

Note: events are currently only emitted to the webhook Url for topics that are implemented in the rest api. We listen for them in the server.ts file.

I think we can close this now @TimoGlastra


Wrong version in release 3.0.488

The latest version of the migrator (3.0.488) reports its version as 0.0.1 and suggests upgrading to 3.0.488.
Jira Exporter v0.0.1
Copyright (c) Solidify
Wow, there's a newer release out (v3.0.488)! We recommend downloading it for latest features and fixes.

The same issue with the 3.0.489 version.

wi-import.exe has the same behavior.


Is there any way to load css chunks on demand in shadow DOM in production mode?

I am trying to encapsulate whole react application inside the Shadow DOM. But when I build the react application, CSS Chunks are loading in  tag. I am wondering, if there is any configuration to load your css files into particular HTML element or Shadow DOM in webpack.

Any updates on this?

@kishori-patil1 Did you ever find any solutions for this? I'm also building a react application where we create a shadow dom as the child of the react 'root' element. I'm also hoping to find a solution that allows me to write styles with CSS (technically Sass) modules and without ejecting CRA.


Better handling of sector images

Right now sector images are handled in a pretty crude way, just being sector x side x track raw images. This is good enough when reading, as the geometry is autodetected from the physical media but it's not good enough when writing --- e.g. the Brother writer will want to know whether to write a 40 track or 80 track disk.
There are, however, a jillion different non-raw disk formats. (.TD0, .IMD, .RAW, .DSK, .DMK, .IMG...) Maybe I should support something like libdsk to abstract over these.

From talking with John Elliott, it sounds like libdsk isn't the greatest at handling non-uniform disk images, which I seem to have a disproportionate number of. It also doesn't like non-power-of-two sector sizes (which I also have a disproportionate number of). So I'm thinking that libdsk and LDBS maybe aren't the right choice here. Maybe I just need another sqlite file...


Update failed

Version: 1.0.90
Platform: win32 10.0.18362
Plugins: none
The process is killed. Restart is still the old version.

I had the same experience trying to update from alpha 88; watching the list in Process Explorer, it seems like the setup process itself is also killed when the main Terminus process is closed.

Yep, I can confirm this has been going on for many versions now, including .90 to .91 today.
If you allow the installer to start the actual installation process (progress bars and all), quickly close the lingering Terminus console window yourself, the installer should finish; it will prompt if you want to relaunch Terminus - go ahead if you like, it should be updated fine this time.
Seems it all goes wrong if you leave the existing console window open and both terminate without updating.

Edit: Should mention this is for my installations set for All Users. I wonder if folks using the user profile (Only For Me) installation do not have this experience, since privileges are more relaxed there. I have yet to test that install scheme.

FWIW, I install mine with the "Just for me" option, so it's not affected by that part..
Also, Process Explorer shows a tree view of processes, with child processes listed in a collapsible section of it's parent process. From that I can tell that (in this scenario) the setup package is launched as a child process of the primary Terminus instance, after that I suspect the setup package is killing the target process tree (i.e.: target process and all it's children) when it closes Terminus.
Perhaps the solution here is to get the setup package to be launched as it's own top-level process instead? Then it wouldn't commit suicide when it closes the currently running Terminus instance.

I can confirm that the problem appears in the "Just for me" installation also. Here's my situation:
Version: 1.0.104
Platform: win32 10.0.18363 (x64)
Plugins: none
Install folder: c:\users\username\AppData\Local\Programs\Terminus

Program starts, after a few seconds gift icon appears
Click on gift icon
Click on "Update"
Select "Just for me" and click on "Install"
Click "Ok" on pop-up warning that program is still alive
Both Terminus and Terminus Update Process exit after a few seconds.

End


Generated from 2c8803b07958009dfbcedab279a1f7b7511acab5 (#4161)

Remove the AzureDataLakeStoreCosmosStructuredStreamDataset since it's internal only

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] The PR targets the latest branch.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

Can one of the admins verify this patch?


Zooz ZST10 700 after firmware update to 7.17.1 still with nodes dropping off network

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
Spontaneously at times but also when multiple Zwave commands are sent (ex. a Home Assistant Automation to turn off all switches at midnight.
What do you observe?
The nodes fail and show as dead and the commands sent to them fail or occur many seconds to minutes later.
Device information
Manufacturer: Zooz
Model name:  ZST10 700
Node ID in your network: 1
How are you using node-zwave-js?

[ ] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[X] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version:
node-zwave-js branch: 8.11.2
zwavejs2mqtt branch: 6.5.0
Did you change anything?
yes (please describe)
If yes, what did you change?
Upgraded firmware to 7.17.1 per #3906
This was worse with the 7.15 firmware but even with 7.17.1, it seems that 1+ node is always dead (incorrectly) and multiple nodes will drop off at the same time when multiple zwave commands are sent.
Did this work before?
No, it never worked anywhere
If yes, where did it work?
No response
Attach Driver Logfile
Attaching logs but truncating/combining from 2/5/22 11:00pm - 2/6/22 1:00am (Eastern) [2022-02-06T04:00:00 - 2022-02-06T06:00:00].
Lots of errors:

[Security2CCMessageEncapsulation] [INVALID]
error: Duplicate command
Dropping message with invalid payload
Message authentication failed, cannot decode command. Requesting a nonce...
[Security2CCNonceGet] [INVALID]
No SPAN is established yet, cannot decode command. Requesting a nonce...
the node is unknown or not initialized yet, discarding...
The node did not respond after 3 attempts, it is presumed dead
Timed out while waiting for a response from the node (ZW0201)

I don't really know where to go from here.
zwavejs_2022-02-05.log
I'll email my network key for decoding

Does rolling back to zwavejs2mqtt 6.0.3 (zwave-js 8.8.3) resolve your dead nodes issue?

So my USB Sticks are attached to a 10ft USB Extension cord and moved far from any other electronics.
That being said, I've had a multitude of other problems since I opened this issue. I attempted to move my NVM settings from a Zooz ZST10 700 to a Zooz ZST10 (not 700). However, it looks like the ZST10 I got was DOA or defective and wouldn't connect to any nodes. It wouldn't connect after restoring the 700 series NVM nor after factory resetting and attempting to pair from a blank stick. Unfortunately, after I moved back to the ZST10 700 somehow this also wouldn't connect to any nodes.
All of this started because my Nortek combo stick failed. Now, I have a Nortek Combo stick that is failed, a Zooz ZST10 700 that is flaky, and a ZST10 that is DOA. Not having good luck.
I have another ZST10 (not 700) coming in the mail today and will hopefully be able to use this until the 700 series issues are better sorted out. I will be starting from scratch with this stick coming in today and re-pairing my ~90 devices.
Thus, I'm not going to be able to help for now in debugging the 700 series issues. Sorry and thanks!


Add Python checkers and linters into BDD repository

Add Python checkers and linters into BDD repository

Implemented via https://github.com/RedHatInsights/insights-behavioral-spec/pull/79


replace @total_ordering by implementing all comparison operators

Fixes #10526
since using @total_ordering has performance tradeoffs, I've replaced @total_ordering decorator by implementing the remaining comparison operators  __eq__(), __ne__(), __lt__(), __le__(), __gt__(), and __ge__()

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x]  Run make lint from the Wagtail root.

This looks like the same as #10545 (PR already up)
@infraredCoding does this PR add anything different to the PR that's already in progress?

@lb- oh my bad. I didn't notice the prev PR before making my own 😅 sorry about that.

OK thanks I will close this one then @infraredCoding
Have a read of our guide for your first contribution, it even has a checklist for finding a good issue.
https://docs.wagtail.org/en/stable/contributing/first_contribution_guide.html#checklist-for-a-candidate-issue

@lb- thanks for the doc reference. I'll keep this in mind. Sorry for the trouble.

No problems at all, good on you for giving it a go, hopefully you find something else to try.
If you want, there is this older PR it has unit tests & docs, just needs to be rebased/re-done on the current main branch and get reviewed.
https://github.com/wagtail/wagtail/pull/5744#issuecomment-1619861507
Be sure you understand how to reproduce the scenario and the problem.

@lb- yeah sure. Thanks a lot. I'll give it a try


Feature Request: migrate from gym to gymnasium

Hi, would it be possible for iGibson to be upgraded from gym to gymnasium? Gymnasium is the maintained version of openai gym and is compatible with current RL training libraries (rllib and tianshou have already migrated, and stable-baselines3 will soon).
This repository is currently listed in the gymnasium third party environments but we are cleaning the list up to only include maintained gymnasium-compatible repositories.
For information about upgrading and compatibility, see migration guide and gym compatibility. The main difference is the API has switched to returning truncated and terminated, rather than done, in order to give more information and mitigate edge case issues.

+1 this would be super helpful, we're starting to run into issues in using igibson with RL libraries


Change primary and secondary colors

🛠 Fixes Issue
Closes #128

👨💻 Changes proposed

Changes the values of the primary and secondary variables in the main stylesheet
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

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


Enhanced Functionality and Styling of Cart Page

Description
A clear and concise description of what the PR does.

This PR does the following:
-Functionality added to display error on loading , Empty cart, and selected items conditionally
-Styled them accordingly and separately

Fixes #547
Screenshots (if applicable)

https://github.com/user-attachments/assets/5870fff8-3b3c-4325-9d3f-9552bb5b2200
This is the dummy content I created to test my changes, it still has increase n decrease functionality in quantity
Additional Context
Kindly add all the tags while merging PR.
Thank you for assigning me this issue.
If you liked my contribution, kindly upgrade my level because I Not only enhanced UI but worked on functionality too.
Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] I am working on this issue under GSSOC

@Bhum-ika nice work 👍 ! Are you sure your changes don't generate any new warnings ? ( you haven't filled the checkbox in the PR template)

@Trisha-tech
It will not generate new warnings, I have checked the tickbox now.
Could you please add level 3 here. I worked hard on this.
I would really appreciate it

@Trisha-tech Kindly reply

@Trisha-tech Kindly reply

Hey @Bhum-ika ! The PA might be busy and she will merge them once she gets time. Till then I'd suggest you to look into other issues.


Shabbir Solution

@jaybobo

LGTM but remember to add .DS_Store to your global gitignore file; it'll make your life easier. :)


Trouble Connecting to MongoDB

Hi Guys,
Love your tutorial. I'm stuck on Ch.2. I can't get the "test" database to show up when connecting to MongoDB. I've read the other issues on here and tried the solutions, but no luck. My terminal says the following when I run yarn dev-express:
=====
yarn run v1.12.3
$ nodemon server/app.js --watch server
[nodemon] 1.19.1
[nodemon] to restart at any time, enter rs
[nodemon] watching:
/Users/jonathangrossman/Documents/Developer/BuilderBook/builderbook/book/2-begin/server/**/*
[nodemon] starting node server/app.js

Ready on http://localhost:8000
[ event ] client pings, but there's no entry for page: /
[ event ] build page: /
[ wait ]  compiling ...
=====

It never comes off "[wait] compiling" and my MongoDB account shows 0 databases and 0 collections. I've posted my code here: https://github.com/jonegrossm/2-begin
I'd really appreciate any help you have!
Thank you.
Jon

Click to see Hill for issue #224

Click to see Hill for all issues

created by Async

I had a breakthrough! Instead of putting the mongoDB connection string in my .env file, I put it in app.js file to see what happened. In app.js, I put it here: const MONGO_URL="connection string."
It worked! The "test" database showed up in mongoDB. So I'm pretty sure I've connected it. However, as you know better than me, I don't want to leave my mongoDB connection string in the app.js file. So I moved everything back to the way your tutorial has it setup, and it seems to be working. The user's email from the mongoDB database shows up on the index page. I ran the various tests in the tutorial to ensure proper connection, and passed them all.

@jonegrossm Good to hear. So no problems so far? I think that dotenv works as expected by the end of the chapter 2.

So far I'm good. I'm halfway through chapter 3. Thanks for following up.
On Sun, Aug 4, 2019 at 11:10 PM Tima<EMAIL_ADDRESS>wrote:

@jonegrossm https://github.com/jonegrossm Good to hear. So no problems
so far? I think that dotenv works as expected by the end of the chapter 2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/builderbook/builderbook/issues/224?email_source=notifications&email_token=AF6JQBJ7BGYVY7WWFNFGKC3QC6R23A5CNFSM4IJFUNIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3QUAXY#issuecomment-518078559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF6JQBOUHYJWNUU4FMCPMV3QC6R23ANCNFSM4IJFUNIA
.


Doesn't work for China regions

Doesn't work for China regions. Is there any work-around to make it working for CN?

hi @reznid01, be aware I am not using this anymore, also I have no experience with China regions.
I see the AWS Java SDK has this note

The default AWS Security Token Service (STS) endpoint ("sts.amazonaws.com") works for all accounts that are not for China (Beijing) region or GovCloud. You only need to change the endpoint to "sts.cn-north-1.amazonaws.com.cn" when you are requesting session credentials for services in China(Beijing) region or "sts.us-gov-west-1.amazonaws.com" for GovCloud.

The code currently hardcodes sts.amazonaws.com in https://github.com/andreabedini/aws-credentials-from-saml/blob/25ec9f9aa0ccdc25abbe1a9341467d6df5cdf814/background.js#L35, perhaps one could try to change that to sts.cn-north-1.amazonaws.com.cn.
Let me know if it works,
Andrea


Unable to login using admin/admin

Hi,
I have setup RasaUI with the latest code but I am unable to login using the console.
Am I missing something?
Thanks

Most cases it would be issue with node modules on web. Did you run npm install from within web/src directory? Can you check if there are any errors on the browser console.

I think it was the nginx server which was not letting me login. Using it directly on port 5001 works.
Thanks


Lighthouse

For use with the "Lighthouse" Places preset

HFC Symbol

Standalone symbol designs:

32
24
16


stream_edit: Move set_stream_property to new stream_settings_api module

(78, 35) → (77, 36). Discussion.

Cool, yeah this seems like progress, merged, thanks @andersk!


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-27-main" workflow

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

Questions ?
Please refer to the Backport tool documentation


Auto relay join

If there is a relay found on MDNS, use it, if not join  moq.quicr.io

On the esp32 , need some code like
https://docs.espressif.com/projects/esp-idf/en/v4.4/esp32/api-reference/protocols/mdns.html
@suhasHere


Refused to apply style from 'http://localhost:3000/style.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

hi all,
i try to reproduce this example<EMAIL_ADDRESS>on my machine and i got this error message in my chrome browser
 Refused to apply style from 'http://localhost:3000/style.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

On Microsoft Edge i got 404 not found for style.css file !
https://afnay.com/images/nodechaterro/chaterror.PNG
https://afnay.com/images/nodechaterro/chaterror2.PNG
https://afnay.com/images/nodechaterro/chaterror3.PNG
https://afnay.com/images/nodechaterro/chaterror4.PNG
 npm version 5.6.0
 ejs: 2.6.1
 express: 4.16.4
 nodemon: 1.18.6
 socket.io: 2.1.1

I thought the problem was coming from ejs, but using HTML with consolidate and mustache, I had the same problem.
 <link rel="stylesheet" href="styles.css" type="text/css">     **Produces Error & no style !!!**
 <link href="styles.css" type="text/css">      **No error & no style !!!**

what wrong  !?
thank you

Can you make sure that you have your style.css placed in your public directory and have express.static('public') enabled in your server script.
And any questions like such are better to be asked on https://gitter.im/expressjs/express


Add character: <6> to font:

Character :
Font :

what file is this for?

unassigned as inactive

Character : 6
Font : carlos

Assign me this

@im0vishal  i am assigning it to you .


RAS: Basic SBE Halt error injection & Recovery (Boot time and Runtime)

Scope of this Automation will include following

Inject SBE halt error during IPL time and expect recovery until threshold is exceeded (Perform this on master and slave SBE)
Inject SBE SEEPROM error injection during IPL time and expect recovery until threshold is exceeded  (Perform this on slave SBE) . Also need to include check for the SEEPROM side switch as part of expectation

Repeat #1 and #2 @ BMC/Host runtime and check for the SBE recovery at Host runtime (Before threshold) - Additional expectation is to check if the expected error logs with correct FAPI RC for SBE failure is captured in the error logs

@lkammath IPL is an IBMism, can you use more industry-standard terminology ? boot?

Done @gtmills


a ProgressDialogType.Download bug for 1.2.3

just 1.2.3
test code:
import 'package:flutter/material.dart';
import 'package:progress_dialog/progress_dialog.dart';

class ProgressPage extends StatefulWidget {
  @override
  _ProgressPageState createState() => _ProgressPageState();
}

class _ProgressPageState extends State<ProgressPage> {

  @override
  Widget build(BuildContext context) {
    ProgressDialog pr =ProgressDialog(
        context,
        type: ProgressDialogType.Download
    );
    pr.style(
//      message: 'Downloading file...',
      message:
      'Lets dump some huge text into the progress dialog and check whether it can handle the huge text. If it works then not you or me, flutter is awesome',
    );
    return  Scaffold(
      appBar: AppBar(
        title: Text("progress"),
      ),
      body: ListView.builder(
        padding: new EdgeInsets.all(15.0),
        itemExtent: 50.0,
        itemBuilder: (BuildContext context,int index){
          return  InkWell(
            onTap: (){
              pr.show();
            },
            child: Text("content: $index"),
          );
        },
      ),
    );
  }
}

error:
I/flutter ( 4108): ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter ( 4108): The following assertion was thrown during performLayout():
I/flutter ( 4108): RenderFlex children have non-zero flex but incoming height constraints are unbounded.
I/flutter ( 4108): When a column is in a parent that does not provide a finite height constraint, for example if it is
I/flutter ( 4108): in a vertical scrollable, it will try to shrink-wrap its children along the vertical axis. Setting a
I/flutter ( 4108): flex on a child (e.g. using Expanded) indicates that the child is to expand to fill the remaining
I/flutter ( 4108): space in the vertical direction.
I/flutter ( 4108): These two directives are mutually exclusive. If a parent is to shrink-wrap its child, the child
I/flutter ( 4108): cannot simultaneously expand to fit its parent.
I/flutter ( 4108): Consider setting mainAxisSize to MainAxisSize.min and using FlexFit.loose fits for the flexible
I/flutter ( 4108): children (using Flexible rather than Expanded). This will allow the flexible children to size
I/flutter ( 4108): themselves to less than the infinite remaining space they would otherwise be forced to take, and
I/flutter ( 4108): then will cause the RenderFlex to shrink-wrap the children rather than expanding to fit the maximum
I/flutter ( 4108): constraints provided by the parent.
I/flutter ( 4108): If this message did not help you determine the problem, consider using debugDumpRenderTree():
I/flutter ( 4108):   https://flutter.dev/debugging/#rendering-layer
I/flutter ( 4108):   http://api.flutter.dev/flutter/rendering/debugDumpRenderTree.html
I/flutter ( 4108): The affected RenderFlex is:
I/flutter ( 4108):   RenderFlex#47123 relayoutBoundary=up9 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE(creator: Column ← Padding ← Expanded ← Row ← Column ← Padding ← Container ← _Body ← DefaultTextStyle ← AnimatedDefaultTextStyle ← _InkFeatures-[GlobalKey#95448 ink renderer] ← NotificationListener<LayoutChangedNotification> ← ⋯, parentData: offset=Offset(0.0, 0.0) (can use size), constraints: BoxConstraints(w=404.0, 0.0<=h<=Infinity), size: MISSING, direction: vertical, mainAxisAlignment: start, mainAxisSize: min, crossAxisAlignment: center, verticalDirection: down)
I/flutter ( 4108): The creator information is set to:
I/flutter ( 4108):   Column ← Padding ← Expanded ← Row ← Column ← Padding ← Container ← _Body ← DefaultTextStyle ←
I/flutter ( 4108):   AnimatedDefaultTextStyle ← _InkFeatures-[GlobalKey#95448 ink renderer] ←
I/flutter ( 4108):   NotificationListener<LayoutChangedNotification> ← ⋯
I/flutter ( 4108): The nearest ancestor providing an unbounded width constraint is: RenderFlex#7321c relayoutBoundary=up6 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE:
I/flutter ( 4108):   creator: Column ← Padding ← Container ← _Body ← DefaultTextStyle ← AnimatedDefaultTextStyle ←
I/flutter ( 4108):     _InkFeatures-[GlobalKey#95448 ink renderer] ← NotificationListener<LayoutChangedNotification> ←
I/flutter ( 4108):     CustomPaint ← _ShapeBorderPaint ← PhysicalShape ← _MaterialInterior ← ⋯
I/flutter ( 4108):   parentData: offset=Offset(0.0, 0.0) (can use size)
I/flutter ( 4108):   constraints: BoxConstraints(264.0<=w<=504.0, 0.0<=h<=838.0)
I/flutter ( 4108):   size: MISSING
I/flutter ( 4108):   direction: vertical
I/flutter ( 4108):   mainAxisAlignment: start
I/flutter ( 4108):   mainAxisSize: min
I/flutter ( 4108):   crossAxisAlignment: center
I/flutter ( 4108):   verticalDirection: down
I/flutter ( 4108): See also: https://flutter.dev/layout/
I/flutter ( 4108): If none of the above helps enough to fix this problem, please don't hesitate to file a bug:
I/flutter ( 4108):   https://github.com/flutter/flutter/issues/new?template=BUG.md
I/flutter ( 4108):
I/flutter ( 4108): The relevant error-causing widget was:
I/flutter ( 4108):   Column
I/flutter ( 4108):   file:///E:/flutter_src/flutter_windows_v1.12.13+hotfix.5-stable/flutter/.pub-cache/hosted/pub.flutter-io.cn/progress_dialog-1.2.3/lib/progress_dialog.dart:225:36
I/flutter ( 4108):
I/flutter ( 4108): When the exception was thrown, this was the stack:
I/flutter ( 4108): #0      RenderFlex.performLayout.<anonymous closure> (package:flutter/src/rendering/flex.dart:691:11)
I/flutter ( 4108): #1      RenderFlex.performLayout (package:flutter/src/rendering/flex.dart:718:10)
I/flutter ( 4108): #2      RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #3      RenderPadding.performLayout (package:flutter/src/rendering/shifted_box.dart:206:11)
I/flutter ( 4108): #4      RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #5      RenderFlex.performLayout (package:flutter/src/rendering/flex.dart:806:17)
I/flutter ( 4108): #6      RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #7      RenderFlex.performLayout (package:flutter/src/rendering/flex.dart:744:15)
I/flutter ( 4108): #8      RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #9      RenderPadding.performLayout (package:flutter/src/rendering/shifted_box.dart:206:11)
I/flutter ( 4108): #10     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #11     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #12     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #13     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #14     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #15     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #16     _RenderCustomClip.performLayout (package:flutter/src/rendering/proxy_box.dart:1232:11)
I/flutter ( 4108): #17     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #18     RenderConstrainedBox.performLayout (package:flutter/src/rendering/proxy_box.dart:259:13)
I/flutter ( 4108): #19     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #20     RenderPositionedBox.performLayout (package:flutter/src/rendering/shifted_box.dart:392:13)
I/flutter ( 4108): #21     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #22     RenderPadding.performLayout (package:flutter/src/rendering/shifted_box.dart:206:11)
I/flutter ( 4108): #23     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #24     RenderPadding.performLayout (package:flutter/src/rendering/shifted_box.dart:206:11)
I/flutter ( 4108): #25     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #26     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #27     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #28     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #29     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #30     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #31     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #32     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #33     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #34     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #35     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #36     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #37     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #38     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:105:13)
I/flutter ( 4108): #39     RenderOffstage.performLayout (package:flutter/src/rendering/proxy_box.dart:3168:13)
I/flutter ( 4108): #40     RenderObject.layout (package:flutter/src/rendering/object.dart:1724:7)
I/flutter ( 4108): #41     RenderStack.performLayout (package:flutter/src/rendering/stack.dart:505:15)
I/flutter ( 4108): #42     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:1584:7)
I/flutter ( 4108): #43     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:844:18)
I/flutter ( 4108): #44     RendererBinding.drawFrame (package:flutter/src/rendering/binding.dart:344:19)
I/flutter ( 4108): #45     WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:774:13)
I/flutter ( 4108): #46     RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:283:5)
I/flutter ( 4108): #47     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1102:15)
I/flutter ( 4108): #48     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1041:9)
I/flutter ( 4108): #49     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:957:5)
I/flutter ( 4108): #53     _invoke (dart:ui/hooks.dart:259:10)
I/flutter ( 4108): #54     _drawFrame (dart:ui/hooks.dart:217:3)
I/flutter ( 4108): (elided 3 frames from package dart:async)
I/flutter ( 4108):
I/flutter ( 4108): The following RenderObject was being processed when the exception was fired: RenderFlex#47123 relayoutBoundary=up9 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE:
I/flutter ( 4108):   creator: Column ← Padding ← Expanded ← Row ← Column ← Padding ← Container ← _Body ← DefaultTextStyle
I/flutter ( 4108):     ← AnimatedDefaultTextStyle ← _InkFeatures-[GlobalKey#95448 ink renderer] ←
I/flutter ( 4108):     NotificationListener<LayoutChangedNotification> ← ⋯
I/flutter ( 4108):   parentData: offset=Offset(0.0, 0.0) (can use size)
I/flutter ( 4108):   constraints: BoxConstraints(w=404.0, 0.0<=h<=Infinity)
I/flutter ( 4108):   size: MISSING
I/flutter ( 4108):   direction: vertical
I/flutter ( 4108):   mainAxisAlignment: start
I/flutter ( 4108):   mainAxisSize: min
I/flutter ( 4108):   crossAxisAlignment: center
I/flutter ( 4108):   verticalDirection: down
I/flutter ( 4108): This RenderObject had the following descendants (showing up to depth 5):
I/flutter ( 4108):     child 1: RenderConstrainedBox#e379d relayoutBoundary=up10 NEEDS-PAINT
I/flutter ( 4108):     child 2: RenderFlex#17729 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108):       child 1: RenderParagraph#27a5d NEEDS-LAYOUT NEEDS-PAINT
I/flutter ( 4108):         text: TextSpan
I/flutter ( 4108):     child 3: RenderConstrainedBox#494d2 NEEDS-LAYOUT NEEDS-PAINT
I/flutter ( 4108):     child 4: RenderPositionedBox#f2113 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108):       child: RenderParagraph#d9650 NEEDS-LAYOUT NEEDS-PAINT
I/flutter ( 4108):         text: TextSpan
I/flutter ( 4108): ════════════════════════════════════════════════════════════════════════════════════════════════════
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderFlex#47123 relayoutBoundary=up9 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderPadding#d58fb relayoutBoundary=up8 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderFlex#f2aa0 relayoutBoundary=up7 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderFlex#7321c relayoutBoundary=up6 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderPadding#0e61f relayoutBoundary=up5 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: _RenderInkFeatures#34484 relayoutBoundary=up4 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderCustomPaint#dbdd6 relayoutBoundary=up3 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderPhysicalShape#c15eb relayoutBoundary=up2 NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
I/flutter ( 4108): Another exception was thrown: 'package:flutter/src/rendering/shifted_box.dart': Failed assertion: line 321 pos 12: 'child.hasSize': is not true.
I/flutter ( 4108): Another exception was thrown: RenderBox was not laid out: RenderPhysicalShape#c15eb relayoutBoundary=up2

can you please use the latest flutter version

can you please use the latest flutter version

my use flutter v1.12.13+hotfix.5  and v1.17.1 , Both errors occurred in version 1.2.3

can you please use the latest flutter version

I encountered the same problem, and my flutter version is the latest.

Same here, just ping me if I can help you out with debugging.

Same here

Same error :((

can someone give me a piece of code to reproduce this issue

I found the same error today, espically when I use the ProgressDialogType.Download ，So I change it to ProgressDialogType.Normal,  I think maybe we use the ListView cause this error.
my code as below :
 pr = ProgressDialog(context,
        type: ProgressDialogType.Normal, isDismissible: false, showLogs: true);

await dio.download(appLink, apkFilePath, onReceiveProgress: (rec, total) {
        downloadingRate = rec / total;
        downloadingRateString =
            "Downloading ..." + ((rec / total) * 100).toStringAsFixed(0) + "%";
        pr.update(message: downloadingRateString);
      });

I found the same error today, espically when I use the ProgressDialogType.Download ，So I change it to ProgressDialogType.Normal, I think maybe we use the ListView cause this error, because the total height is dynamic .
my code as below :
 pr = ProgressDialog(context,
        type: ProgressDialogType.Normal, isDismissible: false, showLogs: true);

await dio.download(link, path, onReceiveProgress: (rec, total) {
        downloadingRate = rec / total;
        downloadingRateString =
            "Downloading ..." + ((rec / total) * 100).toStringAsFixed(0) + "%";
        pr.update(message: downloadingRateString);
      });

Sorry guys, actually I was involved in some other stuff, I will resolve this asap. Thanks a ton all of you!


Add Open Graph image for Showcase and Builder Tools section

Add Open Graph image for Showcase and Builder Tools section since currently portal displays landing page by default

builds OK and I can see the metadata tags in the source... in fact they're valid image URLs, because I guess these 2 image files were already there? 😎

Correct :)


Please invite me to the Recode-Hive GitHub Community Organization

Name
Rhea
Discord Username (if applicable)
No response
Additional Context
I want to explore the Data Science track.

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


Add Unit Tests for Core library

The task is to

[x] Add unit tests for Request Builder
[x] Add unit tests for Response Handler class
[x] Add unit tests for Local DateTime helper
[x] Add unit tests for Zoned DateTime helper
[x] Add unit tests for Date helper
[x] Add unit tests for Core helper
[x] Add unit tests for Test Helper

This sub-task is done and dusted.


New Module Request: Neverwinter Nights Enhanced (Steam)

Is anyone already working on, or could we get an AMP module to host a Neverwinter Nights Enhanced dedicated server?

This request does not follow the template


Optical estimation

The specular and diffuse reflectivity estimation works for simple cases.
Note: the setParameterValues set the maximum value to 1. The absorbtivity is not updated, such that the analytical/numerical partials work (unitTestAccelerationPartials)

Hi, this pull request is not yet finished right (?) From what I remember, there was an additional term missing in the partials when normalizing the sum of the coefficients to 1


Change one-shot node parameter for correctly root motion

Fixes #40294

I just tried it and noticed that this fix does not solve the problem in current. The bug is still occurring and I will look into it later.


Add Automated Required Validation

The field definition's isRequired property should automatically trigger required validation.

Closed via d59a3bcfd87f5c328125c03a1f3495c2d17b65ff


translation in datapack

hey, i have try to translate the en_us file in the mod.jar
after that i do this to the datapack in assets/tinkerslevelingaddon/lang/de_de.json
but ingame it doesnt translate. is the language code wrong?
Edit: Okay no, if i put the json file directly in the jar, then it works. i have make it right.
but why it dont work with the datapack?

Hi,
Translations belongs in resource packs, not datapacks.

ah okay i understand, then...must i put some more in an ressourcepack anless the mcmeta file and the lang file?