[Help] HTTP Error 429

Hello,
i got this error while i'm using youtube-dl only from YOUTUBE:
ERROR: Unable to download webpage: HTTP Error 429: Too Many Requests (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
PS: i'm using the last update of youtube-dl (2016.09.19)
please help 😄

See https://github.com/rg3/youtube-dl#http-error-429-too-many-requests-or-402-payment-required and #5138.


Tweaks to make init-config work all the time

Resources in init-config were being mishandled if your ambassador_id or AMBASSADOR_CONFIG_BASE_DIR were changed from their defaults. Obviously that's a problem -- this fixes it.

Merging for now, but I think I'll switch this per @LukeShu's excellent suggestion later.


Javascript functionality for the quotes

Added an array of quotes
Added query selectors for button and message
Created event listener to add quote to text

Great job!

Wow! Amazing!


Dont't include ignored packages when adding a changeset

Fixes #436
Filters ignored packages before displaying the list of packages when adding a new changeset.

I'd like to see this get merged in soon. Having to skip over sample apps in our mono repo every time we publish is becoming a chore

@Andarist Any chance of getting this merged in?

Also a ping from me. Please merge it.

I would also find this extremely useful and consider it a blocker for fully adopting changesets in @starbeamjs/starbeam

Same here. Anything that can be done to get this merged? :) @Andarist thanks

@Andarist is there something I can do to help merge this?

At this point, would it be appropriate to temporarily fork and publish this with the changes?

At this point, would it be appropriate to temporarily fork and publish this with the changes?

This is one of happiest thumbs-downs I've ever received on Github 😄

@wycats https://github.com/changesets/changesets/pull/871 was already opened to fix the bug.  Somehow I missed the not in my original PR, but there is already a PR to fix it.

@Andarist I just verified the fix applied in 2.23.2 works correctly.  Thanks for the quick resolution to my mistake!

@mskelton I apologize if my tone sounds aggressive or annoying. I'm just really excited about the prospect of building changesets into the Starbeam release workflow, so I'm eager (overeager?) to help 😳.
I already did a handful of releases using Changesets, and I'm cautiously optimistic that with this bugfix (and a few minor tweaks to how we're using it now, including figuring out how to get the exact automation we need for PRs) that we'll be using it for years to come.
Thanks so much for your engagement 🙌


Bump oauth version

note $subject

Merging this PR without running PR builder as PR builder is ran against https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/2413 and no other PR merge during


move shouldReplaceOnComparisonTie to base class to be more reusable

no new changes added, but moved shouldReplaceOnComparisonTie() method to base class to make it more reusable in other subclasses

cc @rohityadav1993

Codecov Report
Attention: Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (581062a).
Report is 596 commits behind head on master.

Files
Patch %
Lines

...cal/upsert/BasePartitionUpsertMetadataManager.java
0.00%
12 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13353       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2472       +36
  Lines        133233   136261     +3028
  Branches      20636    21165      +529
=============================================
- Hits          82274        0    -82274
- Misses        44911   136261    +91350
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


MSL: Add support for op OpCompositeInsert in OpSpecConstantOp?

Several Vulkan CTS tests use op OpCompositeInsert in OpSpecConstantOp, which is not currently supported in SPIRV-Cross. Consequently, these tests fail in MoltenVK.
I've had a look, but it is not at all clear to me how a modified constant composite can be created in MSL via SPIRV-Cross. The corresponding instruction for the non-constantOpCompositeInsert op relies on emitting statements to crate a temp var copy of the composite, and then modifying part of it, which won't be possible with a constant value.
For a constant expression, it would seem we will have to construct the new composite on the fly, with the appropriate element modified as the composite is being created, but unfortunately, the internal plumbing for that is beyond my understanding of SPIRV-Cross innards.

I suppose it's possible to implement OpCompositeInsert as a Construct, where one of the elements are replaced. It gets very hairy with things like spec constant arrays though, since we don't necessarily know the array size in SPIRV-Cross compile time, but MSL doesn't support array spec constants to begin with ...

but MSL doesn't support array spec constants to begin with

Yes. WRT array lengths, that specific issue is probably something we can leave for a later revision or enhancement, but for now we need to live with that general restriction about spec constants under MSL.


WIP H.265 decoder

This is not ready yet.
Notably, it is missing:

Tests
The actual VA-API backend code.
The cros-libva integration
Negotiation support

OTOH, the DPB management  looks pretty much done to me.

argh..

I think you are seeing this error because you are based on a pretty old commit - the generic parameter has probably been added after and since the CI merges against main the error is hit.
If you rebase on top of main you will see it - I do at least. Thankfully this should be easy to fix if you follow what H.264 does.


NOP

@scotthasbrouck
I will delete this branch in 7 days because it's a bit outdated. Let me know if you think otherwise.

i think we already merged this in (with a rebase though), just forgot to delete the brnach


Update docs to explain how the SDK namespaces are used

Requirement: Service partners want to understand how the SDK namespaces are utilized in the release process.  They are unsure if there are todos for them or if the SDK team is handling. If SDK team is handling, they still want to understand where these namespaces are utilized in the release process.
Email exchange in progress with Jesse, Laurent, Renhe, Ray, Mayuri, Allen Z, and Mariana.

Moving request to this issue:

@jsquire or someone form the .NET team needs to provide a link to their docs where the information about adding a new namespace prefix is.

@lirenhe Is helping us understand where we can find docs about how/where to use an approved package namespace:

Management plane
Data plane

from @lirenhe :
For mgmt. SDK, more details could be found in files under azure-rest-api-specs/documentation/code-gen at main · Azure/azure-rest-api-specs (github.com)
e.g. [azure-rest-api-specs/documentation/code-gen/configure-go-sdk.md at main · Azure/azure-rest-api-specs (github.com)]
The description for DPG related configuration is out of date which need to be updated.  [azure-rest-api-specs/documentation/onboard-dpg-in-sdkautomation at main · Azure/azure-rest-api-specs (github.com)]

@lirenhe :

Management plane: I see that not all languages have a document with the instructions. Is this something that is going to be improved during Dt?
Data plane: Who is in charge of updating that doc? and do you have an ETA?

@lirenhe :

Management plane: I see that not all languages have a document with the instructions. Is this something that is going to be improved during Dt?
Data plane: Who is in charge of updating that doc? and do you have an ETA?

We have the task in Dt for documentation update, I will create separate issues and assign the owners.

For the missing Java and .NET part in mgmt. SDK, probably this is by design.
@weidongxu-microsoft  @live1206,  please help to confirm.  thanks

namespace is included in SDK review process https://azure.github.io/azure-sdk/policies_reviewprocess.html

This purely informational/educational session is to let the Azure SDK Architecture board get up to speed with the service and the library/new features that are coming. This allows for early feedback and will potentially affect the service design. High level topics such as API namespaces, function names, and types will be suggested in this first discussion.

for mgmt, that usually be an async review only for namespace.

https://eng.ms/docs/products/azure-developer-experience/develop/namespace-review is about the SDK namespace review guide.

Thank you @weidongxu-microsoft  and @live1206 . What our users have been wondering, is once their namespace is approved, where should they go and update it. Renhe was sharing how currently they need to do it in the Readme.md files, so I am looking for the documentation that says that and that will be applied for all the languages

For Java, the current process is that SDK dev must manually check the namespace review issue and confirm it been approved.
There is no way to retract a released Java package, hence the manual verify step.
Will update in the documentation task.
We'd hope that the namespace review issue would be integrated to the release request (or self-serve) that has this verification built-in.

We'd hope that the namespace review issue would be integrated to the release request (or self-serve) that has this verification built-in.

This is great. But how do users know where the namespace should go?
For example, looking at the Python documentation, I see that in the python.readme.yml file, there is a property namespace and package-name . Should both be updated? is this also applicable for the other languages?
For example, I don't see it in Go, .NET, Java

Currently Java is in SDK repo (automation tool would update the file)
https://github.com/Azure/azure-sdk-for-java/blob/main/eng/mgmt/automation/api-specs.yaml
This is Java only.
(it has less rows than the full spec, because it only register the case that namespace is not same as folder name in spec -- e.g. it would ignore the row "folder cognitiveservices to namespace com.azure.resourcemanager.cognitiveservices")

PR to update onboard doc
https://github.com/Azure/azure-rest-api-specs/pull/28740
For data-plane, only provide doc for typespec.
For management-plane, usually nothing need to be done for Java (besides release request).


Display Line Numbers for fenced Code Blocks

Is it already possible to show line numbers for fenced code blocks? I couldn't find anything about it in the docs. Might be a good feature, if it is not possible yet.

@ZelphirKaltstahl please use pygments to highlight and add line numbers.


fresh k3s install k3s-server process high cpu and failed to response to kubectl

Environmental Info:
K3s Version: v1.20.0+k3s2

Node(s) CPU architecture, OS, and Version: Linux k0 5.4.83-v7l+ #1379 SMP Mon Dec 14 13:11:54 GMT 2020 armv7l GNU/Linux

Cluster Configuration: 1 master, 5 workers. All raspberry pi 4.

Describe the bug:
Fresh installation afterwards on the server node, k3s-server would stay on around 100+% cpu and it fails to answer any kubectl command.

Steps To Reproduce:

uninstalled old k3s installation using "k3s-uninstall.sh" (https://rancher.com/docs/k3s/latest/en/installation/uninstall/).
find . -type d | grep rancher | xargs rm -Rf (manually remove any directory on the system that has the word "rancher" on it. )
follow the install doc  to make sure the node uses legacy iptables.
curl -sfL https://get.k3s.io | sh -
k3s --debug kubectl get nodes     <--- would simply hangs..

Installed K3s: curl -sfL https://get.k3s.io | sh -

Expected behavior:

k3s kubectl get nodes <--- should returns at least showing master node before I register any worker nodes.
k3s-server shouldn't consistently on high cpu usage.

Actual behavior:

k3s-server didn't start properly
k3s-server failed to communicate with kubectl.

Additional context / logs:

journalctl.log - log tail while I run the k3s server installation
k3s-install-screen-shot.txt - the screen that shows my installation command and the output from the screen
k3s-kubectl-hangs.txt - run with --debug, but not much output

journalctl.log - log tail while I run the k3s server installation
k3s-install-screen-shot.txt - the screen that shows my installation command and the output from the screen
k3s-kubectl-hangs.txt - run with --debug, but not much output

Downgrade to 1.19 or wait till 1.20.2, There is a known issue with v1.20 on arm caused by a golang compiler issue. v1.20.2 will include a workaround: kubernetes/kubernetes#97685

Downgrade to 1.19 or wait till 1.20.2, There is a known issue with v1.20 on arm caused by a golang compiler issue. v1.20.2 will include a workaround: kubernetes/kubernetes#97685

Watching k3s-server with top consistently shows below 10% usage on my raspberrypi 4 using commit 6e768c301e77738ffd69934363aa0479c4a516d6. I am also able to execute kubectl commands successfully, including deploying, removing, etc.


NETOBSERV-1554 Add CLI to Krew index

Description
This is a preparatory work for krew index creation:

[X] make command unique since krew is accepting a single bin entry
[X] add make release target to generate krew package and prepare github release
[x] detect when CLI is running from krew to force either kubectl or oc

Here is a krew index sample:
https://gist.github.com/jpinsonneau/a34785c85a20c3899ec91f266ff33823
You can test it using:
kubectl krew install --manifest=netobserv-cli-index.yaml

The krew index will be automatically generated by github release job and appened to body release notes as draft.
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[X] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[ ] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[X] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/lgtm


construct each handler by a custom backend

Closes #10676
Briefly describe your proposed changes:
Rebasing and continuing #11037 work on backends for HTTP handlers (@zhulongcheng).

[x] Rebased/mergeable
[x] Tests pass
[x] http/swagger.yml updated (if modified Go structs or API)
[x] Sign CLA (if not already signed)

Notice this PR needs a twin one in IDPE for compatibility.


The Box Model in CSS

Description 📜
Every element is surrounded with a box. Here I will explain The Box Model in CSS via Video.

Domain of Contribution 📊

[x] Frontend Dev (HTML, CSS, JS)

/assign

/assign

Hi @code-reaper08! I want to work on this issue in the form of a document. Kindly assign me!
Thank you :)

Hey @code-reaper08, I've created a PR for the assigned issue. Please review it :)

@code-reaper08 PLEASE OPEN THIS ISSUE. I HAVE ALREADY MADE A VIDEO BUT COULDN'T CONTRIBUTE AT THAT TIME DUE TO SOME FAMILY PROBLEMS. PLEASE LET ME CONTRIBUTE VIA VIDEO HERE. I DON'T WANT MY EFFORTS IN THE VIDEO TO GO WASTE. ALSO, THE ISSUE WAS OPENED JUST 4 DAYS AGO.PLEASE OPEN THIS ISSUE AGAIN !!

@2024-SANDHYA Video contribution request, reopening the issue here.

@2024-SANDHYA Video contribution request, reopening the issue here.

Great !

@code-reaper08 Hi,Please review my PR. Thankyou !


Add flag to clean schema before running migrations

Fix for #2
This PR adds a new flag to run Flyway.clean() if it's enabled by the user. The flag is disabled by default.
We don't use Flyway FluentConfiguration.cleanDisabled configuration property because by default is false so we would delete users' schemas by default!
@jameskleeh I've used the Pair class you introduced for FlywayEndpoint and moved to a common package. I'm not sure if that's the best place or if it's better to have it duplicated as private class in two places.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Misc Enhancements

What kind of change does this PR introduce?

Adds new Auth Providers
Fixes #38

What is the current behavior?

Missing Slack & Spotify Providers
Missing keys in User.toJson()

What is the new behavior?

Includes Slack & Spotify
Adds missing keys - phone, phone_confirmed_at, emailed_confirmed_at to User.toJson()
Removed old key - confirmed_at - from User.toJson()

Additional context
N/a

Any estimate to when this will be merged?

Sorry @DriesCruyskens for taking our time. I will fix this and try to get it merged ASAP.


Add a number of apps that have built in nord themes to the website

Add apps that have the nord theme built in to the website, under ports, maybe under a section called unofficial ports. I know fish, bitwarden, joplin, and bat.  On the website there are 33 oficial ports, I would say if we count themes that are built into apps, there will be more like 45, maybe more, that people looking on the website dont know exists.

Cool
Regards Chinmay M.R.
@.***
On Mon, Oct 18, 2021 at 12:15 AM hellosway @.***> wrote:

Joplin, Bitwarden, Fish, Lualine (neovim status line), Bat, kitty (with
built in theme kitten), there is a few more that dont come to mind.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/arcticicestudio/nord/issues/161#issuecomment-945174653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQPOWNVBQUOUBJ6CLP3JIELUHMKUZANCNFSM5FW7KACA
.

wezterm has a nord theme: https://wezfurlong.org/wezterm/colorschemes/n/index.html#nord


MIssing References GetNamedCatalogsAsync() - #66 VS2022

I've upgraded my extension to 2022, and visual studio finds it and I can step into it.
However, During runtime I get this message box:
image
Is this related?
@BertanAygun
Member
BertanAygun commented 7 days ago
@ssemon, can you confirm if you also updated the ProjectSystem.Sdk package as mentioned above? This sounds like a mismatch of package versions.

It's occuring when updating my vs2019 project to the latest ProjectSystem.Sdk package

The image didn't come through, can you copy paste the error?

[VS2022] Various ProjectProperties methods and Properties are now missing.... #6

6

It was stated that there may be a mismatch in packages.  So I updated the packages in a visual studio 2019 project and the same issue occurred.   The next step was to update the Microsoft.VisualStudio.Shell.Framework package and I get:
Could not install package 'Microsoft.ServiceHub.Analyzers 3.0.3078'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.7.2', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

Do you have a reference to meta SDK package in your project: https://www.nuget.org/packages/microsoft.visualstudio.sdk, this should make sure all SDK references are in sync so you shouldn't need individual references to Shell.Framework and so on.

Hi Bertan, I tried it and it complains about the same thing.
Install-Package : Could not install package 'Microsoft.ServiceHub.Analyzers 3.0.3078'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.7.2', but the package does not contain any assembly references
or content files that are compatible with that framework. For more information, contact the package author.
At line:1 char:1

Install-Package Microsoft.VisualStudio.SDK -Version 17.0.31902.203

  + CategoryInfo          : NotSpecified: (:) [Install-Package], Exception
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.InstallPackageCommand

Install-Package : Could not install package 'Microsoft.ServiceHub.Analyzers 3.0.3078'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.7.2', but the package does not contain any assembly references
or content files that are compatible with that framework. For more information, contact the package author.
At line:1 char:1

Install-Package Microsoft.VisualStudio.SDK -Version 17.0.31902.203

  + CategoryInfo          : NotSpecified: (:) [Install-Package], Exception
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.InstallPackageCommand

Could not install package 'Microsoft.ServiceHub.Analyzers 3.0.3078'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.7.2', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.
Time Elapsed: 00:00:00.7530641

Hi Bertan,
Ok, I started over and followed your instructions.
I was able to step into the Extension and see that it works up to a point.
I'll file a new issue asking if we can still use c++ and com for a debug engine, or if we need to update to a csharp implementation.
I'll close this now.
Thanks,
Stuart


Render error on Windows 10 x64

I've successfully installed the package after setting up node-gyp and successfully recorded a session, however, the render command fails almost instantly:

Have end the recording session before start the rendering? Please try play command too to check the output

Playing works and the YML file is there. I've stopped the recording with "exit".

Cool, it seems a bug I feel that I know the reason. Will be fixed within 2 hours.

Looking forward to it

I've commented out line 17 out of render/index.js and now it works:
//app.dock.hide();

It seems that you'd have to verify the OS before hiding the Dock.

@petrepopescu21 the fix is released, please update your package and confirm that it is fixed

Perfect!


RPM packaging

Add RPM packaging to the repo, and update pghoard / myhoard etc to depend on it.

Done in #25


"Invalid context" error message is unhelpful when mixing paths with globs

Checks

[X] I understand project setup issues should be asked on StackOverflow or in GitHub Discussions.
[X] I updated to latest http-proxy-middleware.

Describe the bug (be clear and concise)
This message:
ERR_CONTEXT_MATCHER_INVALID_ARRAY = '[HPM] Invalid pathFilter. Expecting something like: ["/api", "/ajax"] or ["/api/**", "!**.html"]',

doesn't explain that mixing globs and non-globs is prohibited. Although it shows one array with just non-glob paths and one with globs, it doesn't clarify that these are examples of 2 separate categories of pathFilters.
I had to find the answer by Googling the error message and reading the code that produces it.
Step-by-step reproduction instructions
createProxyMiddleware(['/api/**', '/foo'], {
    target: "https://httpbin.org",
    changeOrigin: true,
  })

→ https://codesandbox.io/p/devbox/http-proxy-middleware-forked-6ky4n8?file=%2Fsrc%2Findex.js%3A10%2C5
Error message:
[HPM] Invalid pathFilter. Expecting something like: ["/api", "/ajax"] or ["/api/**", "!**.html"]
<stack trace>

Expected behavior (be clear and concise)
Error explaining that globs and paths cannot be mixed. Perhaps just this explanation added on top of the existing message.
[HPM] Invalid pathFilter. Expecting something like: ["/api", "/ajax"] or ["/api/**", "!**.html"]. Plain paths (e.g. "/api") can't be mixed with globs (e.g. "/api/**").

How is http-proxy-middleware used in your project?
Indirectly, with Webpack's devServer.

What http-proxy-middleware configuration are you using?
[irrelevant]

What OS/version and node/version are you seeing the problem?
[irrelevant]

Additional context (optional)
The error is thrown in this line.
The only cases handled for array are pathFilter.every(isStringPath) and pathFilter.every(isGlobPath). isStringPath checks that the string is not a glob - so for mixed paths and globs neither of the conditions will be fullfilled.

Hi @mcFrax
Thanks for suggestion. Sounds like a good improvement.
Could you create a PR with your suggestion to improve the error message?


Some arrows in the generated puml are pointing in the wrong direction

I think that all <|-- arrows should point in the other direction --|>

You are correct, @maxhbr. I'll fix this.

Fixed in https://github.com/spdx/spec-parser/commit/2e8a8c6942810af732a461dd107e63e43cdc87ed


Add support for Ingress's newly-released layers for explorer and pioneer activity

The generic Ingress Intel map now shows a player's visited (explorer) and captured (pioneer) portals using two filters: https://community.ingress.com/en/discussion/13451/portal-history-now-available-on-intel-map
Currently, this feature is not working in IITC. Could you add a feature for these layers? Thanks!

Bump!

This Redditor made a plugin that works in IITC CE. I couldn't get it working in IITC.me.
https://github.com/EisFrei/IngressPortalHistoryFlags

Related: https://github.com/IITC-CE/ingress-intel-total-conversion/issues/434

Implemented in IITC-CE.


HttpPostRequestDecoder.cleanString() puts "32" instead of " "

Netty version: current master (3d6819623efc2f634009094ac8cf6a362fd3c2c1)
Steps to reproduce:
Use the following test case:
    public void testFilenameContainingSemicolon() throws Exception {
        final String boundary = "dLV9Wyq26L_-JQxk6ferf-RT153LhOO";
        final DefaultFullHttpRequest req = new DefaultFullHttpRequest(HttpVersion.HTTP_1_1, HttpMethod.POST,
                "http://localhost");
        req.headers().add(HttpHeaderNames.CONTENT_TYPE, "multipart/form-data; boundary=" + boundary);
        // Force to use memory-based data.
        final DefaultHttpDataFactory inMemoryFactory = new DefaultHttpDataFactory(false);
        final String data = "asdf";
        final String filename = "tmp;0.txt";
        final String body =
                "--" + boundary + "\r\n" +
                        "Content-Disposition: form-data; name=\"file\"; filename=\"" + filename + "\"\r\n" +
                        "Content-Type: image/gif\r\n" +
                        "\r\n" +
                        data + "\r\n" +
                        "--" + boundary + "--\r\n";

        req.content().writeBytes(body.getBytes(CharsetUtil.UTF_8.name()));
        // Create decoder instance to test.
        final HttpPostRequestDecoder decoder = new HttpPostRequestDecoder(inMemoryFactory, req);
        assertFalse(decoder.getBodyHttpDatas().isEmpty());
        InterfaceHttpData part1 = decoder.getBodyHttpDatas().get(0);
        assertTrue(part1 instanceof FileUpload);
        FileUpload fileUpload = (FileUpload) part1;
        assertEquals("tmp 0.txt", fileUpload.getFilename());
        decoder.destroy();
    }

The reason is that HttpConstants.SP is of byte type, so instead of sb.append(HttpConstants.SP) it should be sb.append((char) HttpConstants.SP))

Maybe submit a PR?

Am 21.07.2015 um 15:39 schrieb dop251<EMAIL_ADDRESS>Netty version: current master (3d68196)
Steps to reproduce:
Use the following test case:
public void testFilenameContainingSemicolon() throws Exception {
    final String boundary = "dLV9Wyq26L_-JQxk6ferf-RT153LhOO";
    final DefaultFullHttpRequest req = new DefaultFullHttpRequest(HttpVersion.HTTP_1_1, HttpMethod.POST,
            "http://localhost");
    req.headers().add(HttpHeaderNames.CONTENT_TYPE, "multipart/form-data; boundary=" + boundary);
    // Force to use memory-based data.
    final DefaultHttpDataFactory inMemoryFactory = new DefaultHttpDataFactory(false);
    final String data = "asdf";
    final String filename = "tmp;0.txt";
    final String body =
            "--" + boundary + "\r\n" +
                    "Content-Disposition: form-data; name=\"file\"; filename=\"" + filename + "\"\r\n" +
                    "Content-Type: image/gif\r\n" +
                    "\r\n" +
                    data + "\r\n" +
                    "--" + boundary + "--\r\n";

    req.content().writeBytes(body.getBytes(CharsetUtil.UTF_8.name()));
    // Create decoder instance to test.
    final HttpPostRequestDecoder decoder = new HttpPostRequestDecoder(inMemoryFactory, req);
    assertFalse(decoder.getBodyHttpDatas().isEmpty());
    InterfaceHttpData part1 = decoder.getBodyHttpDatas().get(0);
    assertTrue(part1 instanceof FileUpload);
    FileUpload fileUpload = (FileUpload) part1;
    assertEquals("tmp 0.txt", fileUpload.getFilename());
    decoder.destroy();
}

The reason is that HttpConstants.SP is of byte type, so instead of sb.append(HttpConstants.SP) it should be sb.append((char) HttpConstants.SP))
—
Reply to this email directly or view it on GitHub.

Let me fix this

Apologies, could not get to creating a PR. I think there is another bug, this method should not be used to clear uploaded file names, because it replaces commas which is a valid character for a file name. I'll probably open another issue...

@dop251 can you check https://github.com/netty/netty/pull/4036 ?

Looks good to me, except I would change ";" (which is allowed in a file name) to ":" (which isn't) in the test case, so that when it's fixed properly the test case doesn't break.


Listview with  Image.network throwing exception from another children widget when pass to new Screen

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
To reproduce the issue, please copy and run the code in sampe code.
Expected results
If I tap the child widget, it passes the data as arguments to the new screen. and should not include data on other children.
for example : When we tap on the second child, which has a valid image source, it will navigate to a new screen and will display the provided image.
and in the details screen, the image loaded is just the data passed from navigation (i.e. data[2] )
Actual results
but it seems that when we navigate, the first and third data are also passed to the new screen. and in my example, the first child has an invalid image source, and that causes an exception. (see demo video)
Code sample
Code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const NotePage(),
    );
  }
}

class NoteModel {
  final String imgSrc;
  final String name;

  NoteModel({this.imgSrc = "", this.name = "Lorem Ipsum dolor sit amet"});
}

List<NoteModel> dummyNote = [
  NoteModel(imgSrc: "https://invalid-source"),
  NoteModel(imgSrc: "https://placehold.co/600x400.png", name: "Lorem 2"),
  NoteModel(imgSrc: "https://placehold.co/600x400.png", name: "Lorem 3"),
  NoteModel(imgSrc: "https://placehold.co/600x400.png", name: "Lorem 4"),
  NoteModel(imgSrc: "https://placehold.co/600x400.png", name: "Lorem 5"),
];

class NotePage extends StatelessWidget {
  const NotePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: Column(mainAxisAlignment: MainAxisAlignment.center, children: [
          SizedBox(
            height: 130,
            child: ListView.builder(
              scrollDirection: Axis.horizontal,
              itemCount: dummyNote.length,
              itemBuilder: (context, index) => SizedBox(
                width: 230,
                child: InkWell(
                  onTap: () => Navigator.push(
                      context,
                      MaterialPageRoute(
                        builder: (context) => NoteDetailPage(
                          data: dummyNote[index],
                          // key: ValueKey('$index , ${dummyNote[index].imgSrc}'),
                        ),
                      )),
                  child: Card(
                    child: Row(
                      children: [
                        Expanded(
                            child: Image.network(dummyNote[index].imgSrc,
                                fit: BoxFit.cover,
                                semanticLabel: dummyNote[index].name,
                                loadingBuilder: (context, child,
                                        loadingProgress) =>
                                    loadingProgress != null
                                        ? const Center(
                                            child: CircularProgressIndicator())
                                        : child,
                                errorBuilder: errorImageBuilder)),
                        const SizedBox(width: 20),
                        Expanded(child: Text(dummyNote[index].name))
                      ],
                    ),
                  ),
                ),
              ),
            ),
          ),
        ]),
      ),
    );
  }
}

class NoteDetailPage extends StatelessWidget {
  const NoteDetailPage({super.key, required this.data});
  final NoteModel data;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Padding(
        padding: const EdgeInsets.all(18.0),
        child: ListView(
          children: [
            Image.network(
              data.imgSrc,
              errorBuilder: errorImageBuilder,
              loadingBuilder: (context, child, loadingProgress) =>
                  loadingProgress != null
                      ? const Center(child: CircularProgressIndicator())
                      : child,
            ),
            const SizedBox(width: 20),
            Text(data.name)
          ],
        ),
      ),
    );
  }
}

Widget errorImageBuilder(context, error, stackTrace) => const Material(
      color: Colors.purple,
      child: Icon(
        Icons.image, // change to church icon
        color: Colors.white,
      ),
    );

Screenshots or Video
Video demonstration
https://github.com/flutter/flutter/assets/33835308/f84d4397-e515-4887-aff5-0af5355b4258
Logs
Logs
════════ Exception caught by image resource service ════════════════════════════
Failed host lookup: 'invalid-source' (OS Error: No address associated with hostname, errno = 7)
════════════════════════════════════════════════════════════════════════════════

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.6, on macOS 13.0.1 22A400 darwin-arm64, locale en-ID)
    • Flutter version 3.13.6 on channel stable at /Users/gspe/developments/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ead455963c (5 weeks ago), 2023-09-26 18:28:17 -0700
    • Engine revision a794cf2681
    • Dart version 3.1.3
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/gspe/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] VS Code (version 1.83.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] VS Code (version 1.82.3)
    • VS Code at /Users/gspe/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.0.1 22A400 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.117

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @pmatatias
I can't reproduce this issue on both Android and iOS platforms. There is no exception thrown in the output console. Checking this on the latest Flutter stable channel 3.13.9. Could you try upgrading Flutter and confirm?

Thanks for the response. I've upgrade my flutter.
However, this problem still occurs
I have no idea, i test with mac and widows, both same behavior.
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.9, on macOS 13.0.1 22A400 darwin-arm64, locale en-ID)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] VS Code (version 1.83.1)
[✓] VS Code (version 1.82.3)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

Even though I can't reproduce this issue, I see there's an open issue addressing the case you described https://github.com/flutter/flutter/issues/97077.
Closing this issue as a duplicate, so please follow up on it for further updates.


Cannot reference Generated Permission List

Auto-generated permission list is an amazing feature. For real.
The only downside I am experiencing is when I try to reference the generated constants in the API project itself. Makes sense that at build time, the class is not yet available and thus generates build errors.
What is the recommended use / place for the generator? A separate lib? How can I use the Allow.MyConstantPermission in the API project itself?

we've tested it in multi project solutions and haven't seen what you're describing.
could you possibly throw together a small repro and attach here pls?
it's possible I'm not fully comprehending your use case.
thanks!

Will do, currently prepping a release, will have one up by end of week. Thanks for your quick reply

in the meantime, give this project a go.
if it's giving you the same sort of build errors, it could mean something has gone wrong in your dev environment.

i'll close this for now. feel free to reopen with a repro.
thanks!

Your demo-project pointed me in the right direction. All seems to be working now. Thanks!


loosen requirements

might be a better idea to auto pip install (run pip install in python) instead of creating multiple requirements.txt
close #123

@shubhankarunhale can you help resolve this?

@shubhankarunhale can you help resolve this?

Sure, looking into it!


anyUrl generates invalid JSONPath matcher expression

This is happening on the 1.0.4.RELEASE, 1.0.5.BUILD-SNAPSHOT and 1.1.0.BUILD-SNAPSHOT version for sure.  In short, if you use anyUrl() in the response body the generated assertion code for junit is valid Java but invalid JSONPath.
Steps to Reproduce

git clone https://github.com/pfrank13/plugin-dependency-scc.git
git checkout any_url_bug
./mvnw clean install -Pextending && ./mvnw clean install
observe the stack trace

Output
2017-03-22T11:07:02,808 [main] ERROR com.toomuchcoding.jsonassert.JsonAsserter - Exception occurred while trying to match JSON Path<EMAIL_ADDRESS>=~ /((www\.|(http|https|ftp|news|file)+\:\\/\\/)[_.a-z0-9-]+\.[a-z0-9\\/_:@=.+?,##%&~-]*[^.|\\'|\# |!|\(|?|,| |>|<|;|\)])/)]]
com.jayway.jsonpath.InvalidPathException: Could not find matching close quote for ' when parsing<EMAIL_ADDRESS>=~ /((www\.|(http|https|ftp|news|file)+\:\\/\\/)[_.a-z0-9-]+\.[a-z0-9\\/_:@=.+?,##%&~-]*[^.|\\'|\# |!|\(|?|,| |>|<|;|\)])/)]
	at com.jayway.jsonpath.internal.CharacterIndex.indexOfMatchingCloseChar(CharacterIndex.java:84) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.CharacterIndex.indexOfClosingBracket(CharacterIndex.java:113) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.path.PathCompiler.readFilterToken(PathCompiler.java:444) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.path.PathCompiler.readNextToken(PathCompiler.java:135) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.path.PathCompiler.readContextToken(PathCompiler.java:118) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.path.PathCompiler.compile(PathCompiler.java:54) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.path.PathCompiler.compile(PathCompiler.java:69) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.JsonPath.<init>(JsonPath.java:101) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.JsonPath.compile(JsonPath.java:467) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.JsonContext.read(JsonContext.java:149) ~[json-path-2.2.0.jar:2.2.0]
	at com.jayway.jsonpath.internal.JsonContext.read(JsonContext.java:158) ~[json-path-2.2.0.jar:2.2.0]
	at com.toomuchcoding.jsonassert.JsonAsserter.parseJsonPathAsArray(JsonAsserter.java:205) [jsonassert-0.4.7.jar:?]
	at com.toomuchcoding.jsonassert.JsonAsserter.check(JsonAsserter.java:196) [jsonassert-0.4.7.jar:?]
	at com.toomuchcoding.jsonassert.JsonAsserter.checkBufferedJsonPathString(JsonAsserter.java:213) [jsonassert-0.4.7.jar:?]
	at com.toomuchcoding.jsonassert.JsonAsserter.matches(JsonAsserter.java:159) [jsonassert-0.4.7.jar:?]
	at org.springframework.cloud.contract.verifier.tests.ExampleTest.validate_example(ExampleTest.java:32) [test-classes/:?]
...

Generated test line in and: part of the then
DocumentContext parsedJson = JsonPath.parse(response.getBody().asString());
assertThatJson(parsedJson).field("url").matches("((www\\.|(http|https|ftp|news|file)+\\:\\/\\/)[_.a-z0-9-]+\\.[a-z0-9\\/_:@=.+?,##%&~-]*[^.|\\'|\\# |!|\\(|?|,| |>|<|;|\\)])");

Contract DSL:
Contract.make {
    request {
        method 'GET'
        urlPath '/foo'
        headers {
            accept(applicationJson())
            contentType(applicationJson())
        }
    }
    response {
        status 200
        body ([
            url: anyUrl()
        ])
        headers {
            contentType(applicationJson())
        }
    }
}

Meh... My bad then. Sorry about this.

DelegatingJsonVerifiable.escapedHackedJavaText() changes the \' into a ' which is then run through the java escaper to be a \' which is the root of the issue and the malformed json path "Could not find matching close quote for ' when parsing<EMAIL_ADDRESS>=~ /((www.|(http|https|ftp|news|file)+:\/\/)[.a-z0-9-]+.[a-z0-9\/:@=.+?,##%&~-]*[^.|\'|# |!|(|?|,| |>|<|;|)])/)]"
Next step I need to figure out what the JSONPath generated SHOULD be, right now I don't actually know.  I do know simply removing the escapedHackedJavaTest breaks everything =)

Sadly I'm wrong, I had the order messed up.  I am currently thinking it might be something related not to the code generation because that is actually correct \' escapes to pattern ' internally which is correct, yet we are seeing \'.  I think it's whatever escaping is happening during the runtime of the test.

com.toomuchcoding.jsonassert.JsonAsserter.stringWithEscapedSingleQuotes() sees the ' and escapes it with
\'

But since it's already escaped we get
\\'

And that is the bug

ha! I think that I fixed it - https://github.com/spring-cloud/spring-cloud-contract/pull/265 can you check out that branch to see if it's satisfactory for you @pfrank13 ? :P

Verified this works on my end.
Looking at the fix it appears more of a workaround rather than a fix for regexes that are already escaped though.  Unless I diagnosed the problem incorrectly, but surely you're using a vastly different regex =)

Yes and no :P
The escaping was invalid. For example hostname didn't seem to work fine either. So fixing the escaping fixes the problem. OTOH the url pattern was not very valid :P I added quite a few tests to check this out.
Also this PR finally runs the generated code. So I'm 100% sure that the generated test actually passes against the provided JSON. I won't yet merge the PR cause I'll also add some tests by starting a WireMock instance, feeding it with a stub definition and I'll send a request to see if the response is fine. That way we'll have the whole flow checked out.

Excellent point!  Yeah, I think we're all set here.  Do you usually close these or should I?

No, no. It will automatically close once merged to 1.0.x and master. Thanks for checking it out

No, no, thank you!!!  Glad you're back man, hope you're feeling better =)


Find the limit of how large a secret can get

It’s OS-dependent; but the m5.large machine is a pretty decent candidate that I can test it oout.

also add it to documentation.


On off rework

Simplified monitoring state handling client-side. Disabled default auto-fusing which caused timeouts breaking WS connections for new clients connecting to a debugging session with monitoring disabled (to be investigated later).

:+1:


Avoid using @babel/runtime  as a dependency

Describe the bug

While it is expected that tailwind-merge increase bundle size (as said in the docs) and has a cost at runtime, we should minimze it nonetheless.
We should be able to use modern JavaScript syntax and features, as supported by modern browsers, hence, I don't think adding @babel/runtime (for better compatibility? as I understand it) is worth it.
To Reproduce

npm install tailwind-merge
Expected behavior

No dependencies.
Environment

<EMAIL_ADDRESS>

Hey @theoludwig! 👋
tailwind-merge is bundled to be compatible with the browserslist query > 0.5%, last 2 versions, Firefox ESR, not dead, maintained node versions (code) which is the default for most apps.
Do you have a use case where speed matters so much more than compatibility with enough browsers? The tradeoff with not transpiling at all to shave off a few bytes is that the code will break the app for some 10-20% of users.

Hey @dcastil, thanks for your reply!
I understand the desire to maintain broad compatibility, but I would like to present a case for why we should avoid including the @babel/runtime dependency in the tailwind-merge library.

The tradeoff with not transpiling at all to shave off a few bytes is that the code will break the app for some 10-20% of users.

Maybe, but then, for 80-90% of users, they are downloading and executing more JavaScript, than necessary.
Also, I'm not convinced, that removing @babel/runtime, it ill break for 10-20% users, it really depends of the JavaScript features used by tailwind-merge.
As you said: "bundled to be compatible with the browserslist query", "which is the default for most apps.", that's exactly the point, it's not the library developer responsibility, but the app developer responsibility to bundle/polyfill/transpile accordingly to what browsers they want to target. Most frameworks for apps, already handle it, whether it is Vite.js, Next.js or others.
For reference, some comments/articles, that discuss this:

https://github.com/w3ctag/polyfills/issues/6#issuecomment-272647475
https://www.builder.io/blog/stop-polyfilling-fetch-in-your-npm-package
https://www.reddit.com/r/javascript/comments/purc3r/askjs_best_practices_for_polyfills_in_libraries/

Conclusion
In conclusion, while ensuring compatibility is important, it should not come at the cost of performance and maintainability for the majority of users. By not including @babel/runtime and allowing application developers to handle polyfills, we can create a more efficient and maintainable library that performs optimally in modern environments.
Looking forward to your thoughts on this!

That argument makes a lot of sense to me, thanks for bringing it up!
I won't be able to stop transpiling the default bundle because that could break a lot of apps. But I could add another bundle, e.g. esnext that is only converted to JS without adding any polyfills. You would then need to use it like this in your code:
import { twMerge } from 'tailwind-merge/esnext'

There is already a similar bundle like that for people that need the code to be ES5-compatible (https://github.com/dcastil/tailwind-merge/pull/286, https://github.com/dcastil/tailwind-merge/issues/344):
import { twMerge } from 'tailwind-merge/es5'

Would that be something that could work for you?

Yes, possibly, a 'tailwind-merge/esnext'. :smile:
And the dependency @babel/runtime can't be removed?

That bundle wouldn't use @babel/runtime then. That's what I expect. 🤔 So when you import tailwind-merge/esnext, the Babel runtime wouldn't be in your import tree.

@theoludwig just checked the bundle and the normal tailwind-merge bundle doesn't even import @babel/runtime. You'll find it in node_modules/tailwind-merge/dist/bundle-mjs.mjs. It's only imported in the ES5 bundle in node_modules/tailwind-merge/dist/es5/bundle-mjs.mjs.

But I can see how having @babel/runtime as a dependency of tailwind-merge just for the ES5 bundle is annoying. I'll remove the dependency and inline the transform functions into the ES5 bundle since that isn't optimized for code size anyway.


Coroutines won't be destroyed if they are in channel's wait list

This will cause memory leaks because when the Sender, Receiver and SyncSender are going to be destroyed, the Processor may already dead.
Reproduce code:
extern crate coio;

use coio::Scheduler;
use coio::sync::mpsc;

fn main() {

    Scheduler::new().with_workers(1).run(|| {
        let (tx, mut rx) = mpsc::channel();

        for i in 0..100 {
            let (ltx, lrx) = mpsc::channel();
            let name = format!("Coroutine #{}", i);
            Scheduler::spawn(move|| {
                loop {
                    let value = match rx.recv() {
                        Ok(v) => v,
                        Err(..) => break,
                    };
                    println!("{} passing {:?}", name, value);
                    ltx.send(value).unwrap();
                }
            });

            rx = lrx;
        }

        for i in 0..10 {
            println!("Master gives out {}", i);
            tx.send(i).unwrap();
            let value = rx.recv().unwrap();
            println!("Master gets {}", value);
            assert_eq!(i, value);
        }
    }).unwrap();

}

And add println in Coroutine::drop like this
impl Drop for Coroutine {
    fn drop(&mut self) {
        println!("Dropping!!");
        /// ...
}

You may only see 2 "Dropping" print out on the screen. Because when the main function is finished, the main coroutine is going to be destroyed. But who prints the other "Dropping"? I don't know... But all I know is most of them haven't been destroyed properly.

Didn't we want to avoid the need of waking up coroutines by unwinding it's stack in Coroutine::drop()?

No one will trigger it. As you can see in the code, when the main_function is finished, only then tx and rx will be destroyed. BUT the channel (Sender or Receiver) won't tell the other end that it is going to be destroyed. So the coroutine is still waiting in the waiting list, and it also owns the Sender and Receiver so that they won't be destroyed.

Ah... I missed the important bit: The coroutine owns the channel and the channel owns the coroutine... a cyclic dependency!

Indeed. So that's why I add this commit. The Sender and SyncSender, SyncReceiver should wake up all the waiting coroutines.

Yeah I know... As I said: The important bit is the cyclic dependency of "coroutine owns the channel and the channel owns the coroutine". We might want to add this in a comment in the code to prevent similiar errors later on...

Alright.

I recognized a bug. Because I am waking up coroutines in the Drop trait, so what would happen if the coroutine is already running and try to call try_recv again on the Receiver, but the actual underlying Sender is still alive?


Hopefully prevent pull requests for this non-typo

Related to #446, #430, #429, #410, #339, #304, #277, #288, #275, #191, #200.

(I wait with great excitement for the next PR "fixing" this :D)

Hahaha I love it.


add timestamps to pdf export

Currently, the pdf export only shows the date, not the time. It would be good to add also a time stamp.

Thumbs up on this one, really difficult to interpret the context without the timestamp here.
Please note: the pdf is presently an important after-experiment-workaround for the missing outlining and navigation tools and somehow buggy search.


Merge master branch to prTesting1_201910101258 branch

Docs Build status updates of commit 2b1d16f:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 2b1d16f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

split/updateTestCon1.md
:warning:Warning

Details

split/updateTestCon1.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
[Warning] Missing attribute: title. Add a title string to show in search engine results.
Line 1: [Suggestion] H1 is required. Use a single hash (#) followed by a space to create your top-level heading. NOTE: This Suggestion will become a Warning on 11/25/2019.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


⚠ Secret Website 2 has degraded performance

In 2e95566, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 5265 ms

Resolved: Secret Website 2 performance has improved in e672fd2.


CMU server down?

The new server (http://airlab-share.andrew.cmu.edu) might not be working. Anyone else having issues downloading?

Yes，I meet the same problem

Tagging @andrewjong

now I can download

@Zhuliang1 thanks for updating! It works now :)

Hi, I found that the download_train.py runs with time out at present, is it the same with you?

now I can download

Hey bro, can u share your dataset with me. I always see when run download_train.py;
Connecting to airlab-share.andrew.cmu.edu (airlab-share.andrew.cmu.edu)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://airlab-share.andrew.cmu.edu:8081/tartanair/abandonedfactory/Hard/flow_flow.zip [following]
--2023-09-25 07:44:15--  https://airlab-share.andrew.cmu.edu:8081/tartanair/abandonedfactory/Hard/flow_flow.zip
Connecting to airlab-share.andrew.cmu.edu (airlab-share.andrew.cmu.edu)|<IP_ADDRESS>|:8081... failed: Connection timed out.
Resolving airlab-share.andrew.cmu.edu (airlab-share.andrew.cmu.edu)... <IP_ADDRESS>
Connecting to airlab-share.andrew.cmu.edu (airlab-share.andrew.cmu.edu)|<IP_ADDRESS>|:8081... failed: Connection timed out.
Retrying.


[BugFix] Fix bug Json metadata lost fn info

Fixes #28770
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 11 / 11 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/RangePartitionInfo.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/catalog/ExpressionRangePartitionInfo.java
10
10
100.00%
[]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0


Upgrade version of misspell used for macOS Monterey support

Bug Report
There is a panic running the misspell checks on macOS Monterey. This is a known issue with the project.
Filing this issue to track updating once https://github.com/client9/misspell/issues/190 is fixed and so any
others hitting this issue can find that it is something we are aware of.
Expected Behavior

No boom.
Steps to Reproduce the Bug

Screenshots or additional information and context

Environment Details

Build version (tanzu version):
Deployment (Managed/Unmanaged cluster):
Infrastructure Provider (Docker/AWS/Azure/vSphere):
Operating System (client):

Diagnostics and log bundle

@karuppiah7890 throwing this rel-eng's way.
Please let us know if there isn't bandwidth to knock this out in v0.11.0 -- core eng could also pick it up.

I was reminded of this issue today so took a quick look at the upstream issue status. There actually hasn't been any commits merged to this project since 2018, despite quite a few issues and PRs being filed since then. The project may be dead, so we may have to either find an alternative or fork the project if we want to continue using it.


Enable edition 2018

Update the MSRV to Rust 1.41.1 and enable edition 2018.
I'm not sure how we want to merge these MSRV PRs across the stack but here is the rust-miniscript one.
Discussion: https://github.com/rust-bitcoin/rust-bitcoin/issues/510#issuecomment-881686342

I think the MSRV change can go in any order. I would like to get an external contributor #339 PR in first before we address this. That already has ACK from me, but would also like Andrew's ACK(it's a somewhat big change) before we proceed with it.

Sorry, this now conflicts after we got #339 in

Changes in force push:

Rebase on master
Fix commit: 08bce7a Update to use edition 2018 as needed

FTR please do not worry about merge conflicts caused by this PR in relation to any of my other PRs, I'm fine to rebase.

BOOM! The first edition 2018 PR to be merged across the stack, way to go!


Pin minor version of identity dev dependency to avoid re-recording

Currently, the master branch has the identity package in v2 and requires any consuming package to re-record their recordings due to us relying on the msal libraries for our credential implementation.
In #14909, we have laid out the plan to release a v1 update for Identity instead.
Simply changing the version in Identity package will break tests in playback mode for all packages that use Identity in their recordings.
Since re-recording would take considerable amount of time, we want to unblock the work needed to ship v1 of Identity. Therefore, this PR updates the dev dependencies on identity such that the master branch can stay in a state where tests can remain green in the playback mode.
The issue https://github.com/Azure/azure-sdk-for-js/issues/14581 tracks the work needed to update the dev dependency back to the latest version in each package as and when the recordings are updated.

Thanks all!


One marker to multiple categories

Checklist

[X] I've searched through the plugins to make sure this feature isn't already available, or think it shouldn't require a plugin.
[X] I've searched through the current issues to make sure this feature hasn't been requested already.
[X] I agree to follow the Code of Conduct that this project adheres to.

Motivation
I cannot currently have the same marker on multiple layers - I have to have service providers, who provide multiple types of service.
Currently, when I show the same object in multiple categories, it works otherwise, but when I turn off any of layers it belongs to, it disapperars. It has to disappear, when I turn off all layers.
Suggested solution
Make it disappear, when all layers it has been added to, are turned off - not, when any of those is turned off. Make it reappear like now, when any of layers is switched on. Then it behaves like belonging to multiple categories.
Alternatives considered
No response

For alternatives considered - I added the same place several times, under different categories. This makes map considerably slower and adds multiple shadows / darker shadow to marker, when multiple categories are selected.


Set tox to ease testing and building with different platforms

This also should include the changes needed on the gh actions.

Closed by #107


RFC: Add a ui-sref-resolve directive similar to ui-sref-active(-eq)

The purpose of this is a loading state icon on navigation elements while fetching all data through resolve.
If a user clicks a button in the navigation to change the state (to: app.user.profile) the app needs to fetch the profile data first (resolve: { profile: function($http) { $http.get... } }). During the latency of the network request I'd like to have the button pressed () to get an extra class ("btn-loading") to show the user an indicator that the click was registered.
The way this currently has to be done:

  Profile

function Controller($scope, $state) {
  $scope.changingState = false; // simplified here would need one for each button
  $scope.go = function(state) {
    $scope.changingState = true;
    $state.go(state).then(function() { $scope.changeingState = false; })
  }
}

Proposed change:

  Profile

Need something quite similar over here.
+1

This seems like a very specific solution to a very general problem. Why not something more like this? (Warning: off-the-cuff, untested):
app.directive("myLoader", () => {
  return {
    restrict: "A",
    scope: { myLoader: "=", myLoaderClass: "@" },
    link: (scope, element) => {
      scope.$watch("myLoader", (oldVal, newVal) => {
        if (!newVal || newVal === oldVal) return;
        element[0].classList.add(scope.myLoaderClass);
        newVal.then(() => element[0].classList.remove(scope.myLoaderClass));
      });
    }
  }
});

3 lines of directive linking code that aren't hard-coded to anything. In fact, they're completely composable with any other thing that supports promises.
Then in your controller/template you could do:
$scope.transition = $state.go("app.user.profile");

and:
Profile

What I would consider doing is exposing some kind of event hook on uiSref that exposes the promise returned from $state.go(). Would that maybe work for you?

If I am reading your code right I'm pretty sure it would not work as $state.go would go directly to the state and I'd have to wrap had to ignore ui-sref completely.
The possibility of an exposed $scope.go promise would be what is needed.
I'd propose to convert the ui-sref link function to be an actual controller. Then I'd be able to write my ui-sref-resolve and require the controller.
How does that sound?

I did not know what the preferred way is...
Here is a method triggering a event on the current scope. It's not cleaned up yet, but up and running.
Pro:

it introduces just a few lines, easily tested
it enables every parent-scope to handle the event
might solve #456

Contra:

it triggers an extra event that needs documentation

ToDo:

Decide on event name
clean up code

remove controller again
add toState / fromState ( and other useful data )

        var promise = $state.go(state, params, options);
        if(promise) {
          $scope.$emit('$uiSref', {
            element: element,
            promise: promise
          });
        }

Would this be a solution you'd merge?

I think the better solution would be to convert the ui-sref link function to be an actual controller. Then I'd be able to write my ui-sref-resolve and require the controller.

I think this is a good idea, and not necessarily mutually exclusive with the event idea. I'd go for something like this:
diff --git a/src/stateDirectives.js b/src/stateDirectives.js
index 2715927..1b7fb0f 100644
--- a/src/stateDirectives.js
+++ b/src/stateDirectives.js
@@ -134,7 +134,8 @@ function $StateRefDirective($state, $timeout) {
         if ( !(button > 1 || e.ctrlKey || e.metaKey || e.shiftKey || element.attr('target')) ) {
           // HACK: This is to allow ng-clicks to be processed before the transition is initiated:
           var transition = $timeout(function() {
-            $state.go(ref.state, params, options);
+            var $transition = $state.go(ref.state, params, options);
+            if (attrs.uiSrefOn) scope.$eval(attrs.uiSrefOn, { $transition: $transition });
           });
           e.preventDefault();

Btw, sorry, I realize I wasn't using very clear language. When I said 'event', I meant 'event handler' vis-a-vis something like ng-click. With the above, you can do, i.e.:
...

That lets you handle the transition inline with no extra controller code (unless you need it). If that works for you, and you want to write up some tests & docs for it, I'll merge it (otherwise I'm happy to do it myself as soon as I have time, but I'm betting you'll get to it before I do :turtle:).

Anyway, to conclude, if you want to resubmit a separate PR that just factors out the uiSref link function into a controller, that'd be fine, too. Then you'll have a more flexible API for making directives that goes beyond what's available in the core if you need to do really specific stuff.
(Also, finally, please keep whitespace changes in separate PRs as changes are too hard to track otherwise. Thanks!)

I still think that a real event might be better as you can handle it from anywhere, solving the ui-view issue. But this will work for me.
Thanks

@lordnox Patch looks good. Please squash the commits and fix the message per the contributor guideline, i.e. something like:
feat(uiSref): add ui-sref-on to expose transition promise

Fixes #1863

Thanks!

Arg... need to practice my git-fu ... Hope this is like you need it

@lordnox This looks perfect. Open a PR with just that commit and I'll merge it (I closed the other one for you).


Custom field support

We can currently create custom fields by defining a field with 'name' and defining a method 'get_name'. But the 'get_name' resolver doesn't have access to the model or anything else it might want to base the return value off of (like the request context).
E.g.
class Episode(Object):
    name = CharField()
    number = IntegerField()
    long_name = CharField()

    def get_long_name(self):
        return 'Episode {}: {}'.format($name, $number)

We should have access to $name and $number from the model somehow.

So it looks like you can currently do it like:
        return 'Episode {}: {}'.format(self.data.name, self.data.number)

Maybe that's fine or maybe there's a better way of doing it.


Issue/fix build failure due to pyprojecttoml module converter interaction

Description
Fix a testcase that was failing since we added a pyproject.toml file to all modules.
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box
- [ ] Attached issue to pull request

[x] Changelog entry
- [ ] Type annotations are present
[x] Code is clear and sufficiently documented
- [ ] No (preventable) type errors (check using make mypy or make mypy-diff)
- [ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches master in 12eda9156683fd15d7e383c900702924a61cc5d1

Processing #4162.


Add ability to assign maps to mosaics / categories within the warper

Allow editor users to assign maps to mosaics manually. Similar to personal layers on mapwarper.net
Possibly via a chooser interface on the map partial, or probably a better interface on the edit tab.

I think it would be best to handle categories/mosaics in Commons. It may change with metadata in Wikidata, though. We must be able to keep Commons and Warper in sync.

i'm currently looking at what is involved in shifting from Mapwarper.net (thanks to tim for this site that i've been using for a while now) to the wikimedia mapwarper implementation. a key issue for me is use of mosaics to combine things like sanborn map tiles. i'm not too picky about how mosaics are made available but right now the option is simply not available, and a reason to stick with mapwarper.net
there are a number of factors i would like to see worked out; here are some illustrative links:
Rome 1894 Sanborn Insurance Mapset
for this link, there are both tifs and jpgs for the same tile. is this intended or an error? is this subject to cleanup/removal of duplication and how would that subsequently impact any maps in the mapwarper implementation that used "cleaned up" maps?
Tile 22 of the Rome 1894 map set
one feature of the sanborn tiles is that some of them (the ones with bold black lines) combine several map segments, this was evidently done to reduce page count in the bound map volumes, but these tiles need to be partitioned so that the sections can be separately warped, and then separately mapped to mosaics. how might this be done? otherwise, they need to be uploaded to commons multiple times (which i'm used to doing with mapwarper.net). how would you prevent this from being "cleaned up" in commons and breaking things?
Tile 10 in wikimedia mapwarper
One of the challenges of warping sanborn tiles is that they are small and so alignment is challenging due to precision issues in finding reference points. unlike mapwarper.net, the wikimedia implementation of mapwarper does not provide any modern image source, and frequently having a quality aerial image layer makes a huge difference in alignment. OSM alignment is not always reliable, and my current workaround is to edit OSM and improve road alignment, but that leaves me waiting for the OSM layers in wikimedia mapwarper to update, and i have no idea how long that takes.
for context, my current experiments with the 1894 Rome NY Sanborn tiles are directed at mapping the Enlarged Erie Canal in OpenHistoricalMap.
in the absence of better support for precise alignment junctions between tiles are going to look like the attached screenshot from JOSM, where the Black River canal does not line up from tile to tile:


Migrate to flow-typed

Repo: https://github.com/flowtype/flow-typed
Discussion: https://github.com/flowtype/flow-typed/issues/4

Moving to 2.x.x milestone since that package is still in unknown condition.


Add Xcode9 & iOS11 SDK Support?

The framework installed from cocoapods seems not working well (actually even can't pass the compile)
Could you please add support for Xcode9?
(BTW, the readme.md isn't showing correctly, there should be a space between "#" and the word)

My fork seems to work fine so far: https://github.com/skofgar/KGFloatingDrawer maybe that'll help

You can use my fork for Swift 5 ->
https://github.com/yunustek/KGFloatingDrawer


[Enhancement - Policy] Update the look and feel of the policy screen

🙋 feature request

🤔 Expected Behavior

😯 Current Behavior

💁 Possible Solution

🔦 Context

💻 Examples

will make this enhancement

@astra1

can you please see if you can add the icons as in the dashboard for each category

can you try to add the number in the category inline and just make the whole tile clickable


fix webpack2 example for version 2.3.1 - fixes #708

Webpack requires output paths to be absolute since version 2.3.0.
This PR upgrades webpack's version to 2.3.1 and sets the output path to be absolute.

this is a fix for issue #708


feat(database/rds): Add password constraints

Description of your changes
This adds the ability to set additional password constraints for DBCluster.docdb, DBInstance.rds, DBCluster.rds and RDSInstance.database.
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Unit tests.

I wonder if it would be better to just make the default passwords really good (enough for 95% of users), and have the final 5% just pre-generate a password (provider-secret-password? :smile:) with features like this.

However, we do allow our users to specify their own passwords so we need it to be checked.
[...]
Would it still be possible to get theses changes merged?

I think this is a fairly special case, I'd leave it to @haarchri to decide. I think the best practise is to generate a random master password and never expose it to humans, only use it in other tooling to create less privileged users, so this feature is normally not a requirement.
On the other hand, it's not a lot of code inside each controller.

Closing it since this use case is too specific.


show hostname in footer

or modal

https://github.com/phundament/app/blob/master/src/views/layouts/_modal.php#L17 Done.


PRovide a way to actually enable the natvis diagnostics in VS Code

There doesnt seem to be any way to enable the diagnostic output if you're using this with VS Code; I have no idea how to debug natvis issues without this ability.

You should be able to enable natvis dagnostics in your launch.json.
E.g.
{
   "program": ...,
   "natvisDiagnostics": "verbose"
}

See https://github.com/microsoft/vscode-cpptools/blob/8d42c49b77aadb5ae4286f21efcd159601118805/Extension/package.json#L3730

Ah, it needs to be in the logging object, like this:
{
   "program": ...,
   "logging": {
         "natvisDiagnostics": "verbose",
    },
}

That didn't seem to do anything; the natvis is being loaded because I DO see some of the visualizations I programmed, but i'm not seeing anything in the debug console or the terminal window.
nikita@Nikitas-MBP extensions % /usr/bin/lldb --version
lldb-14<IP_ADDRESS>
Apple Swift version 5.8.1 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>.100)

the launch config:

@WardenGnaw could you write, please, in what window logs should appear?


hello from gnark team

hi, given gnark's current architecture, I suspect you'll come up with a wish list at some point to make your life easier -- feel free to create issues in gnark repo / participate in discussions.
Might be of interest to you: https://github.com/ConsenSys/gnark/pull/452 . In particular there is an example on how to build a R1CS without using the frontend; note that the APIs / interfaces are still quite experimental.
func ExampleR1CS_GetConstraints() {
	// build a constraint system; this is (usually) done by the frontend package
	// for this Example we want to manipulate the constraints and output a string representation
	// and build the linear expressions "manually".
	r1cs := cs.NewR1CS(0)

	ONE := r1cs.AddPublicVariable("1") // the "ONE" wire
	Y := r1cs.AddPublicVariable("Y")
	X := r1cs.AddSecretVariable("X")

	v0 := r1cs.AddInternalVariable() // X²
	v1 := r1cs.AddInternalVariable() // X³

	// coefficients
	cOne := r1cs.FromInterface(1)
	cFive := r1cs.FromInterface(5)

	// X² == X * X
	r1cs.AddConstraint(constraint.R1C{
		L: constraint.LinearExpression{r1cs.MakeTerm(&cOne, X)},
		R: constraint.LinearExpression{r1cs.MakeTerm(&cOne, X)},
		O: constraint.LinearExpression{r1cs.MakeTerm(&cOne, v0)},
	})

	// X³ == X² * X
	r1cs.AddConstraint(constraint.R1C{
		L: constraint.LinearExpression{r1cs.MakeTerm(&cOne, v0)},
		R: constraint.LinearExpression{r1cs.MakeTerm(&cOne, X)},
		O: constraint.LinearExpression{r1cs.MakeTerm(&cOne, v1)},
	})

	// Y == X³ + X + 5
	r1cs.AddConstraint(constraint.R1C{
		R: constraint.LinearExpression{r1cs.MakeTerm(&cOne, ONE)},
		L: constraint.LinearExpression{r1cs.MakeTerm(&cOne, Y)},
		O: constraint.LinearExpression{
			r1cs.MakeTerm(&cFive, ONE),
			r1cs.MakeTerm(&cOne, X),
			r1cs.MakeTerm(&cOne, v1),
		},
	})

	// get the constraints
	constraints, r := r1cs.GetConstraints()

	for _, r1c := range constraints {
		fmt.Println(r1c.String(r))
		// for more granularity use constraint.NewStringBuilder(r) that embeds a string.Builder
		// and has WriteLinearExpression and WriteTerm methods.
	}

	// Output:
	// X ⋅ X == v0
	// v0 ⋅ X == v1
	// Y ⋅ 1 == 5 + X + v1
}

Hi! Thanks for this. You hit the spot, this is more or less what I was looking for (I was planning on forking gnark for this because all I needed was internal haha), even though the frontend API is terrific what I need is something more low-level like this.

Thanks @gbotrel this is really useful!


Unable to sign a custom_json operation with an active key

Expected behavior
I was expecting to be able to sign a custom_json operation with an active key the same way I sign a custom_json with a posting key via the tool https://steemconnect.com/sign/
Actual behavior
When we fill out the field "Required Auths" with the account name and sign the transaction with an active key we should be able to broadcast the custom_json operation without having to enter any data in the Required Posting Auths field.
How to reproduce
- go to https://steemconnect.com/sign/
- select Custom Json
- fill in the Required Auths field with your account name this way: ["accountName"]
- leave the field Required Posting Auths empty (as it is only required when using a posting key not an active key)
- fill in the Id field with "test"
- fill in the json fiels with { "test": "test" }
- click Generate Link
- click Try Link
- click Continue
- enter the account name in the Username field
- enter the active key of the account in the Password or key field
- finally click Sign in

The following error appears:

Call stack:
"RPCError: Assert Exception:required_active.size() == 0:
    at new t (https://steemconnect.com/js/app.min.js:67:208461)
    at https://steemconnect.com/js/app.min.js:67:207945
From previous event:
    at o.h [as _captureStackTrace] (https://steemconnect.com/js/app.min.js:53:10223)
    at o._then (https://steemconnect.com/js/app.min.js:53:41599)
    at o.spread (https://steemconnect.com/js/app.min.js:53:40192)
    at Object.S.send (https://steemconnect.com/js/app.min.js:81:208111)
    at Object.S.(anonymous function) [as customJsonWith] (https://steemconnect.com/js/app.min.js:81:209033)
    at t.<anonymous> (https://steemconnect.com/js/app.min.js:81:320698)
    at n (https://steemconnect.com/js/app.min.js:76:421806)
    at Generator._invoke (https://steemconnect.com/js/app.min.js:76:422851)
    at Generator.e.(anonymous function) [as next] (https://steemconnect.com/js/app.min.js:76:421985)
    at n (https://steemconnect.com/js/app.min.js:12:705)
    at https://steemconnect.com/js/app.min.js:12:801"

By checking the developer console I can see that the wrong operation is trying to be broadcasted:
Current operation brodcasted:
[
    "custom_json",
    {
        "required_auths": [
            "accountName"
        ],
        "required_posting_auths": [
            "accountName"
        ],
        "id": "test",
        "json": "{ \"test\": \"test\"}"
    }
]

Operation that should be broadcasted (empty array in required_posting_auths item):
[
    "custom_json",
    {
        "required_auths": [
            "accountName"
        ],
        "required_posting_auths": [],
        "id": "test",
        "json": "{ \"test\": \"test\"}"
    }
]

So, if nothing is entered in the Required Posting Auths field, an empty array should be sent not a copy of the Required Auths field value.
A workaround works though, fill out the field Required Posting Auths with an empty array [] will send an empty array when the transaction is broadcasted and you won't get any error.

Browser/App version: Google Chrome Version 68.0.3440.106 (Official Build) (64-bit)
Operating system: macOs High Sierra 10;13.1

GitHub Account
https://github.com/harpagon210

You need to add the parameter "&authority=active" on the signing url to force user to sign it with his active key.


Service account already exists after retry after failed to set IAM policy

I ran kubeprod on a new cluster in a GCP project where it turned out I didn't quite have all the necessary permissions:
$ kubeprod install gke \
  --email "${GCLOUD_USER}" \
  --dns-zone "${GCLOUD_DNS_ZONE}" \
  --project "${GCLOUD_PROJECT}" \
  --oauth-client-id "${GCLOUD_OAUTH_CLIENT_KEY}" \
  --oauth-client-secret "${GCLOUD_OAUTH_CLIENT_SECRET}" \
  --authz-domain "${GCLOUD_AUTHZ_DOMAIN}"
INFO  Installing platform gke
INFO  Created Google managed-zone "bkpr.mkm.pub"
INFO  You will need to ensure glue records exist for bkpr.mkm.pub pointing to NS [ns-cloud-e1.googledomains.com. ns-cloud-e2.googledomains.com. ns-cloud-e3.googledomains.com. ns-cloud-e4.googledomains.com.]
INFO  Created service account<EMAIL_ADDRESS>ERROR Error: failed to set IAM policy on project "<redacted>" (possibly caused by not having privileges to grant roles/dns.admin role to service account bkpr-edns-bkpr-mkm-pub@<redacted>.iam.gserviceaccount.com): googleapi: Error 403: The caller does not have permission, forbidden

When my friendly admin granted me the necessary permissions I tried to run it again but it tripped over resources created in the previous run:
$ kubeprod install gke \
  --email "${GCLOUD_USER}" \
  --dns-zone "${GCLOUD_DNS_ZONE}" \
  --project "${GCLOUD_PROJECT}" \
  --oauth-client-id "${GCLOUD_OAUTH_CLIENT_KEY}" \
  --oauth-client-secret "${GCLOUD_OAUTH_CLIENT_SECRET}" \
  --authz-domain "${GCLOUD_AUTHZ_DOMAIN}"
INFO  Installing platform gke
INFO  Using existing Google DNS zone "bkpr.mkm.pub"
ERROR Error: rpc error: code = AlreadyExists desc = Service account bkpr-edns-bkpr-mkm-pub already exists within project projects/<redacted>.

Can kubeprod just ignore an existing service account and just use it (or ask if it's ok to proceed, with -y for non-interactive usage)?

I am running into this issue as well in GKE. It would be useful for the bootstrap process to use an existing service account since the SA deletion does not work reliably for me in the UI and can take up to 15 minutes (on my latest try) to take effect.

I run into this when migrating a zone between clusters. My process for migrating is:

Scale external-dns to zero in cluster one
Install kubeprod in cluster two with same zone

However kubeprod fails to install on the second cluster with the below:
ERROR Error: rpc error: code = AlreadyExists desc = Service account bkpr-edns-${ZONE} already exists within project projects/${PROJECT}.

HI @EamonKeane.
In my experience you would not see this issue if you reuse the kubeprod-autogen.json that was generated when BKPR was installed on cluster one. This file has the service account details that already exists and therefore it will not try to re-created it.
What's happening in your case is that since you are not reusing the existing config, BKPR is trying to recreated the service account and errors out because if already exists.
Having said that I think to migrate BKPR from one cluster to another cluster (in the same GKE project), all you should have to do is change the kubectl context to the new cluster and just do kubeprod install in the same directory with the existing kubeprod-autogen.json and kubeprod-manifests.jsonnet files.

But if you still choose to continue on the current path, then you will have to delete the existing service account first. Here's  a snippet of doing that using the gcloud cli.
NOTE: you are requested to understand the changes that these commands will perform before issuing them to make sure you don't delete stuff inadvertently.
# fill in these details
dnsZone=<your-dns-zone>
gcloudProject=<your-gcloud-project-id>
# clean up the service account
gcloudServiceAccount=$(gcloud iam service-accounts list --filter "displayName:${dnsZone} AND email:bkpr-edns" --format='value(email)')
gcloud projects remove-iam-policy-binding ${gcloudProject} --member=serviceAccount:${gcloudServiceAccount} --role=roles/dns.admin
gcloud iam service-accounts delete ${gcloudServiceAccount} --quiet

thanks @sameersbn . I'll examine that more closely next time to ensure I'm using same kubeprod-autogen.json. Can confirm the method of deleting service-account does work as you say.


Multiple Values don't work with Flexible.

data-condition="#a1 < 5 && #a2 > 25 only the 2nd condition works.
data-condition="#a1 == love && #a1 == magic none of them work.
I even tried just one text condition to show, nothing seems to work.
When i tried typing love it shows error in console : $('#a1').val()==love
Uncaught ReferenceError: love is not defined

You should put love in quotes. Like "data-condition="#a1 == 'love' && #a1 == 'magic'". Otherwise javascript thinks that it is a variable. love is not defined means "Sorry, I see no variable called love".

I actually did put it in quotes and it may not show the error but it didnt work, i will try again tomorrow, thanks.

Ok. If it will not work, please provide the whole non-working example with the context. I mean html code with inputs and etc.

Ok yes it was a cache issue, i reset it and it worked but multiple options still don't work
If i add two options even the first one doesn't work.
data-condition="#a1 == 'love' && #a1 == 'magic'

Just replace && with ||. Otherwise you are waiting for a string to be both ‘love’ AND ‘magic’ at the same time.

Great, Thanks :+1:

Welcome:)

Is it not possible to set it like this data-condition = "#a1 == 'love || magic'" ?
That will require creating of own syntax which is not that flexible. You can use javascript functions inside conditions like data-condition = "['love, 'magic'].includes(#a1)". But be careful with browser support: https://caniuse.com/#search=includes


Text Input typing bug

Python: 3.8
OS: android
Kivy: master
Kivy installation method: pip

When you enter text in the text field, after pressing the space bar, it disappears somewhere, if you start erasing it, it appears, but only after erasing the visible characters.
https://user-images.githubusercontent.com/40869738/110664448-745b6d80-81d8-11eb-8b8c-928131a487bc.mp4

https://youtu.be/OCk6OuXq4dA explains the problem.

https://youtu.be/OCk6OuXq4dA explains the problem.

As I understand this is a bug, but how to fix it?

https://youtu.be/OCk6OuXq4dA explains the problem. This is just a reference. Please try that app and report if issue is exactly the same. I have tried that and can confirm that when using keyboard via otg, the problem mentioned in video disappears

I downloaded the app from the video above, the bug is the same

@darpan5552
Uploading Screenrecorder-2021-03-11-21-23-40-771(0).mp4…

https://user-images.githubusercontent.com/40869738/110835925-7abd1880-82b0-11eb-8fcb-5c3bdb76d0f0.mp4

Upon close observation, we can observe that kivy is forgetting its cursor position. Suppose we are at position1 to type 4 characters, then for typing 5th character if we go to position2, all the 4 characters including 5th character comes at position2.

@darpan5552 Yes, this is a textinput bug, I looked at the source code, but I still can't figure out where the error is

@Neizvestnyj May be the bug is in kivy.core.text instead of kivy.uix.textinput

@darpan5552 Don't know (

i can confirm weird behaviors, it still kind of work, if you trust it to do the right thing and continue, but this is not ideal.
https://user-images.githubusercontent.com/22759/112563046-e89d3000-8dd8-11eb-949b-840e2a2aed66.mp4

Hi, I'm sorry for possible spaming, but I'm having a relatable problem https://github.com/kivy/kivy/issues/7545.
Have been able to find some kind of workaround?

Python: 3.8
OS: android
Kivy: master
Kivy installation method: pip

When you enter text in the text field, after pressing the space bar, it disappears somewhere, if you start erasing it, it appears, but only after erasing the visible characters.
bug.mp4

Hi, I'm sorry for possible spaming, but I'm having a relatable problem #7545.
Have been able to find some kind of workaround?

Python: 3.8
OS: android
Kivy: master
Kivy installation method: pip

When you enter text in the text field, after pressing the space bar, it disappears somewhere, if you start erasing it, it appears, but only after erasing the visible characters.
bug.mp4

Hi, I'm sorry for possible spaming, but I'm having a relatable problem #7545.
Have been able to find some kind of workaround?

No, it doesn't work at the moment

@olek5andr
Use kivy==2.0.0
I have observed that it's working much better

Yes, at the moment the problem is still being solved

@olek5andr
Use kivy==2.0.0
I have observed that it's working much better

Using kivy == 2.0.0 is not an option, because when using kivy=2.0.0 with kivymd it is lagging drastically on an Android device. It appears to run much smoother when kivy==master, but then this issue with TextInput arises.
I'm trying to identify the specific commit after which it started, wasn't able to find it yet...

@olek5andr
I am using kivy==2.0.0 with kivymd==0.104.1 and still haven't observed any lagging. May be those application are lot simpler.
https://github.com/kengoon/PyJniusTester
https://github.com/dcindia/matrix-calculator
https://github.com/dcindia/KivyAppUpdater

It seems the problem still exists in version 2.0.0. Also I had encountered another bug that might related to this issue: When pasting long text using gboard auto paste feature the pasted text is incomplete.

any updates !👀

Can someone test if the bug persists if using p4a manually, since i tried it and it was fixed

@Philipmikmik what you mean exactly

@hussam132 well i was working on a app that experienced this error and now after some time i tried python-for-android by itself, but i also changed a lot of things like my os, python version etc. in the time passed and cannot confirm it was that that fixed it. So im asking if someone can test it

@hussam132 im sorry but i dont think thats possible, since its been over half a year. And i also had a problem that made buildozer unusable thats why i used p4a, its possible the error got cleared in that time

Has anyone fixed it yet? (TextInput)

Any updates ? I'm using kivy version 2.0.0 and the bug still appearing.

Any updates ? I'm using kivy version 2.0.0 and the bug still appearing.

No, in 2.1.0 this feature was removed

Python: 3.10.4
OS: android
Kivy: 2.1.0 master
Kivymd: 1.0.2
buildozer: 1.4.0
p4a: 2022.7.20
Kivy installation method: pip (pipenv)
I'm facing the same behavior as @Neizvestnyj described in his/her first post.
Is anybody working on that issue or is a workaround available? @tshirtman Thank you for every small hint.

I saw this behavior already in this environment:
Python: 2.7.2
OS: android
Kivy: 1.9.0
Problem wasn't there at least until android-version 7.1.1, it appeared with newer devices (don't know exactly when it first come up).
For some other reason I upgrades everything:
Python: 3.9.9
OS: android
Kivy: 2.1.0
I was hoping in the meantime this bug has been found and eliminated. But not :(
From my observation there must be some connection to the android-version!
Any hint would be very welcome ...

I wanted to verify, if my statement above is true: I installed new apk (Python: 3.9.9, Kivy: 2.1.0) on my old Sony Z4 with android 7.1.1, and - contradicting to what I said -  bug in TextInput occurs. So: There is no obvious connection to the version of android.
Would like to help in finding a solution - but don't know how ...


Xamarin.Forms Analytics.TrackEvent to Not Use Location Services by Default

Xamarin Analytics.TrackEvent seems to use Location Services by default to get the country code.  I would like this to be off by default and/or have option to turn it off.  Currently Apple is strict about using this and I don't need the location.
So now I have to add purpose string for the user to see. NSLocationAlwaysUsageDescription and NSLocationWhenInUseUsageDescription. Even though I don't need this information about the user, I have to now prompt them.
If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs. Learn more (https://developer.apple.com/documentation/uikit/core_app/protecting_the_user_s_privacy).

@ant81584 I'm sorry, but I don't understand the purpose of the issue you submitted. I don't see any questions suggestions for the docs. Can you please clarify what you need/want?

I'm going to remove the Xamarin app center Analytics.TrackEvent code to see if this resolves the issue with Apple.  I only got this issue as soon as I started using it the xamarin forms Analytics.TrackEvent.
Sorry @annakocheshkova about the duplicate prompted question.  I will update once removing this event code helps the issue from apple sees.

@ant81584 feel free to remove App Center packages completely and see if the warning goes away. I will close the issue here, lets keep track of this issue in Intercom.