[FEATURE REQ] Easier troubleshooting of 403 (Forbidden) when using DefaultAzureCredential

Library or service name.
Azure.Identity
Azure.Security.KeyVault.Secrets
Is your feature request related to a problem? Please describe.
I have recently rolled out new application with Azure.Identity. Some users are experiencing 403 errors which are difficult to troubleshoot:
Status: 403 (Forbidden)

Content:
{"error":{"code":"Forbidden","message":"Caller is not authorized to perform action on resource.\r\nIf role assignments, deny assignments or role definitions were changed recently, please observe propagation time.\r\nCaller: appid=872cd9fa-d31f-45e0-9eab-6e460a02d1f1;oid=d45307dc-69ea-4daf-a1ac-a9ab54f3650e;iss=https://sts.windows.net/33e01921-4d64-4f8c-a055-5bdaffd5e33d/\r\nAction: 'Microsoft.KeyVault/vaults/secrets/getSecret/action'\r\nResource: '/subscriptions/**/resourcegroups/**/providers/microsoft.keyvault/vaults/**/secrets/**'\r\nAssignment: (not found)\r\nVault: **;location=eastus\r\n","innererror":{"code":"ForbiddenByRbac"}}}

Q1: what was the underlying provider of security token? We spent a lot more time than we should have to identify that particular user was running tool on Azure and token was provided by Managed Identity. The fix was easy once we understood cause, but getting there was not.
[edit] Just had another case where developer had VisualStudio Code credential being picked up. Maybe there's a better way to troubleshoot this, but we ended up inspecting DefaultCredential object's private fields in the debugger and thus were able to find which credential it picked up.
Q2: when authentication fails like above, we need means to inspect the token (for example, run it by https://jwt.ms/). Currently there is no way to get specific token that was used for failed operation
Q3: need clear troubleshooting instructions on removing the cached tokens. We currently have situation where shipped version of the app is picking cached token for the wrong tenant. We know the fix, but it requires code change and shipping that code is not easy. To mitigate the issue, we need to know how to clear cached token, but Azure.Identity does not make it easy. There are instructions  to clear underlying MSAL cache, but it's a different layer - I should not need to go to lower level library; and also I could not make that work, maybe because I need to supply the right ClientId, or maybe because Azure.Identity is using it's own caching

@ayeltsov Thanks for reaching out with your questions and feedback. I know diagnosing authentication issues in general can be difficult, and we want to continue to improve this experience in the Azure.Identity library, as well as across the Azure SDK. Your feedback is very appreciated, and it will help us better understand the practical pain points, and ways we can improve the diagnostic experience. I'll try to address your specific questions here with what is possible in the library today, and I welcome any suggestions you have on how this could be improved upon.

Q1: what was the underlying provider of security token? We spent a lot more time than we should have to identify that particular user was running tool on Azure and token was provided by Managed Identity. The fix was easy once we understood cause, but getting there was not.
[edit] Just had another case where developer had VisualStudio Code credential being picked up. Maybe there's a better way to troubleshoot this, but we ended up inspecting DefaultCredential object's private fields in the debugger and thus were able to find which credential it picked up.

This is a common request from users and an area which definitely needs improvement, even if just documenting currently available troubleshooting. One way which you could currently get this information, other than in the debugger, is via our logs other than in the debugger. Every credential in the Azure.Identity library logs event source events on the success or failure of  GetToken or GetTokenAsync, specifically the events GetTokenSucceeded, and GetTokenFailed.  Since logging all SDK events can be rather noisy, here's a small exampl of how you could filter logged events, to specifically see which credentials have been tried, and which credential ultimately succeeded.
static void Main(string[] args)
{
    AzureEventSourceListener listener = new AzureEventSourceListener(IdentityCredentialEventSourceHandler, EventLevel.Informational);

    SecretClient client = new SecretClient(new Uri("https://myvault.vault.azure.net/"), new DefaultAzureCredential());

    KeyVaultSecret secret = client.GetSecret("mysecret");
}

public static void IdentityCredentialEventSourceHandler(EventWrittenEventArgs eventArgs, string text)
{
    if (eventArgs.EventName == "GetTokenFailed" || eventArgs.EventName == "GetTokenSucceeded")
    {
        Console.WriteLine($"{eventArgs.Payload[0]} {eventArgs.EventName}");
    }
}

Here's some sample output from the above example where the AzureCliCredential was ultimately used by the DefaultAzureCredeential
EnvironmentCredential.GetToken GetTokenFailed
ManagedIdentityCredential.GetToken GetTokenFailed
SharedTokenCacheCredential.GetToken GetTokenFailed
VisualStudioCredential.GetToken GetTokenFailed
VisualStudioCodeCredential.GetToken GetTokenFailed
AzureCliCredential.GetToken GetTokenSucceeded
DefaultAzureCredential.GetToken GetTokenSucceeded

Keep in mind that each instance of DefaultAzureCredential will only walk the chain of credentials once, so on subsequent requests you would only see the last two lines. Also if you want more information as to why particular credentials failed you can just log the text parameter, I only wrote the event name and the first event argument, which is contains the specific credential, to increase the readability for demonstration purposes.
We have considered adding an API onto the DefaultAzureCredential and ChainedTokenCredential in the past, but we opted to start with the logging instead. Would the above approach suit your needs? Or would you prefer to see a property so you could programmatically access the credential which was chosen?

Q2: when authentication fails like above, we need means to inspect the token (for example, run it by https://jwt.ms/). Currently there is no way to get specific token that was used for failed operation

Yes unfortunately getting the actual token used in a particular request is fairly difficult. In order to protect customers from accidently leaking tokens into logs we redact this information from all logs and exceptions by default. You could get this information by specifically enabling the Authorization to be logged via the specific clients as below.
SecretClientOptions options = new SecretClientOptions
{
    Diagnostics = { LoggedHeaderNames = { "Authorization" } }
};

SecretClient client = new SecretClient(new Uri("https://dotnettestvault.vault.azure.net/"), new DefaultAzureCredential(), options);

However, to keep your application secure I would STRONGLY ADVISE YOU NOT DO THIS, or if you do enable this only do this in temporarily in a debug environment, and ensure you protect the logs as LEAKING THE LOGS IN THIS CASE WOULD COMPROMISE THE AUTHENTICATED ACCOUNT.
I'm wondering instead if, in the case, the error returned from key vault actually had the information you needed? It contains the application id, account oid, and tenant (in the form of issuer) corresponding to the token the call was authenticated with.
Caller: appid=872cd9fa-d31f-45e0-9eab-6e460a02d1f1;oid=d45307dc-69ea-4daf-a1ac-a9ab54f3650e;iss=https://sts.windows.net/33e01921-4d64-4f8c-a055-5bdaffd5e33d

Is this information not sufficient? If not what other information would you want from the access token to aid with debugging the failure?

Q3: need clear troubleshooting instructions on removing the cached tokens. We currently have situation where shipped version of the app is picking cached token for the wrong tenant. We know the fix, but it requires code change and shipping that code is not easy. To mitigate the issue, we need to know how to clear cached token, but Azure.Identity does not make it easy.

By default each credential in Azure.Identity has its own token cache, so "clearing the cache" is as simple as creating a new credential instance. However, we have introduces a new feature in 1.4.0-beta.4 which allows you to configure credentials to persist their token cache using a new options class TokenCachePersistenceOptions. Examples on how to use this feature can be be found here, though keep in mind the feature is still in beta so the API might change slightly before our GA of 1.4.0.
Currently, with this feature we don't have a way to clear the persisted cache, but it's an interesting request. Could you give me more information on when your application might want to clear the cache? Also would your application only need to remove every account out of the cache, or just tokens for specific accounts?
I hope these answers here help somewhat. Please let me know your thoughts and if you have any other questions related to diagnosing authentication failures.


Complex Schedule not executing correctly

I Have a job that i want to run every hour. Using the C# Scheduler SDK I created a schedule that looks like the following
{
    "Count": null,
    "EndTime": null,
    "Frequency": 2,
    "Interval": null,
    "Schedule": {
        "Days": null,
        "Hours": null,
        "Minutes": [0],
        "MonthDays": null,
        "MonthlyOccurrences": null,
        "Months": null
    }
}

According the documentation this schedule should run every hour on the hour (see {minutes:[0]})
But in Azure, the schedule is actually only executing once a day. The execution date was 11 October 2016. As you can see the next run was scheduled for the next day instead of the next hour - see image below.

Is this a bug?
Note: I specify the hours in the schedule it seems to work.
{
    "Count": null,
    "EndTime": null,
    "Frequency": 2,
    "Interval": null,
    "Schedule": {
        "Days": null,
        "Hours": [0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23],
        "Minutes": [0],
        "MonthDays": null,
        "MonthlyOccurrences": null,
        "Months": null
    }
}

@aaronroney Please identify an appropriate assigneee for this issue

Adding @kithrik for visibility.

@gorillapower, this does not appear to be a bug.  Have you tried specifying frequency as a string (source: https://docs.microsoft.com/en-us/azure/scheduler/scheduler-advanced-complexity#how-to-use-json-and-rest-api-for-creating-schedules)?
{
    "count": null,
    "endTime": null,
    "frequency": "week",
    "interval": null,
    "schedule": {
        "days": null,
        "hours": null,
        "minutes": [0],
        "monthDays": null,
        "monthlyOccurrences": null,
        "months": null
    }
}

@twitchax thanks for the feedback. Yes apologies, that json was a serialization of the scheduler C# object as i was using the Scheduler SDK Library. I have since managed to get the json of the actual http request.
What im trying to replicate is the behaviour described in the documentation

Here is the JSON that actually sent to the API,
"recurrence": {
	"frequency": "day",
	"interval": 1,
	"schedule": {
		"minutes": [15]
	}
}

and the result is below - schedule runs every day 15 minutes pass the start time. Am I missing something? Bearing in mind the frequency cant be hourly in this case.

You can specify the start time to XX:15, and change "recur every" to "1 Hours".

Ok so the only way is to remove the advanced schedule and run it on a frequency of an hour?
Is it possible to have schedule that runs 15 and 30 minutes past every hour?
I think the documentation is misleading as this cant be done how it is described. Instead you either have to implement your solution above or include every hour 0,1,2,3...23 and run the schedule daily.

Unfortunately, I think you have to enter every hour.  However, this issue would be better suited to the Scheduler team, so I can pass it along to them.

Clearing assignees so the Scheduler team can pick this up.

My apologies for the drop on this one.  Is this still an issue?


[CI] Add pre-commit hooks

got a doubt about the utility of
with:
fetch-depth: 0
if you got any clue Raphael

got a doubt about the utility of
with: fetch-depth: 0
if you got any clue Raphael

@Altiire

got a doubt about the utility of
with: fetch-depth: 0
if you got any clue Raphael

fetch depth 0 = you fetch the whole history. for a first time it can be nice but for the others, you can set it to 50 so you only have the 50 last commits

got a doubt about the utility of
with: fetch-depth: 0
if you got any clue Raphael

fetch depth 0 = you fetch the whole history. for a first time it can be nice but for the others, you can set it to 50 so you only have the 50 last commits

so it checks if not secrets were leaked in the 50 last commits ? In the case we merge it to dev it will check all the 50 commits before ?

got a doubt about the utility of
with: fetch-depth: 0
if you got any clue Raphael

fetch depth 0 = you fetch the whole history. for a first time it can be nice but for the others, you can set it to 50 so you only have the 50 last commits

so it checks if not secrets were leaked in the 50 last commits ? In the case we merge it to dev it will check all the 50 commits before ?

if it has run for this PR then the whole history got already checked. if we set the depth to 50, the last 50 commits will be checked every time

I added the pydantic dependency to this MR @oliver-gordon as our current code is not supported by pydantic 2.0


Cannot Connect to 6.x Image in GitHub Workflow

I have been adding tests to Sqitch, including its support for the Exasol engine, here. It's working well for Versions 7.0 and 7.1, but not for 6.x. I am using the 7.x ODBC driver and EXAplus clients. The ODBC connection failure is:
[EXASOL][EXASolution driver]Error in SSL_connect. SSL error code: 5 = SSL_ERROR_SYSCALL    (SQL-HY000) [state was HY000 now 01002]
[EXASOL][EXASolution driver]Exasol was not connected. (SQL-01002)

The error is the same with SSLCertificate=SSL_VERIFY_NONE and without. Note that it works fine (with SSLCertificate=SSL_VERIFY_NONE) connecting to Exasol 7 (example).
I also tried installing the 6.2 ODBC driver, and that returns a different error:
[EXASOL][EXASolution driver]Error in socket read. (SQL-HY000)
[EXASOL][EXASolution driver]Error code was 0 (SQL-HY000) [state was HY000 now 08S01]
[EXASOL][EXASolution driver]Error receiving message from Exasol. Socket failure. (SQL-08S01)

Which makes it seem like it's not running, but the "Initialize Containers" step clearly shows it starting up just fine. Any idea what might be preventing connections to 6.x in GitHub Actions?

@theory , please check my working GitHub action in PyEXASOL repo: https://github.com/exasol/pyexasol/blob/master/.github/workflows/examples.yml
I guess, it is related to the fact that older Exasol Docker versions used port 8888 instead of 8563.
Thank you.

Oh, good to know, thank you, @littleK0i. Got it working in sqitchers/sqitch@a28daae.

Sadly, I find that builds fail a lot when using this approach. I frequently get
[EXASOL][EXASolution driver]Error in socket read. (SQL-HY000)
[EXASOL][EXASolution driver]Error code was 0 (SQL-HY000) [state was HY000 now 08S01]

But I re-run the builds a few times and eventually it succeeds. I don't know if it's because there are port conflicts because the jobs run in parallel or what, but sadly I don't seem to break able to use random port mappings, as reported in actions/runner#1508.

Fixed the issue with the ports; it was my fault. Now using a random port again, but sadly still getting
[EXASOL][EXASolution driver]Error in socket read. (SQL-HY000)
[EXASOL][EXASolution driver]Error code was 0 (SQL-HY000) [state was HY000 now 08S01]

So I guess the ports weren't the issue. I wonder if Exasol isn't fully started up yet, though about 40s elapsed between the container starting and the tests trying to connect in this build. But perhaps there's something should be passed via the --health* options?

This is not an issue for the 7.x images FWIW; they continue to work reliably.

In theory, you may want to try using an explicit "wait" script. It should iterate in a loop, until the connection server responds.
For example: https://github.com/exasol/pyexasol/blob/master/examples/_wait_container.py
(see only connection part, you don't need java part)
It may take a while for Exasol to start, and I've noticed some variation in startup time when using GitHub actions.

Is there something like that in the container already? Would be handy. For example, the gvenzl/oci-oracle-xe image includes a healthcheck.sh script, used in this GitHub action example.

As far as I know, there is nothing inside the container. And even if Exasol devs add something to newer versions, old versions will remain without this feature.
However, Torsten created this repo for integration tests: https://github.com/exasol/integration-test-docker-environment
It has "wait" functionality built-in. But it also has a lot of other stuff, which you may or may not like.


Api v3

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue

This adds Traffic Ops API version 3.0, including all associated documentation and tests.
Also adds a v3 Go client and makes other components specify the v2 client (for compatibility)
Which Traffic Control components are affected by this PR?

CDN in a Box
Documentation
Traffic Control Client (Go)
Traffic Ops

What is the best way to verify this PR?
Run all unit and client/api integration tests.
The following criteria are ALL met by this PR

[x] This PR includes tests
[x] This PR includes documentation
[x] This PR includes an update to CHANGELOG.md
[x] This PR includes any and all required license headers
[x] This PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

@ocket8888 - why is the database label on this PR?

I think because I did that when this was a draft and I was anticipating having to manipulate seeds.sql for capabilities, but that's not actually how that works

@rawlinp Those first two were because of rebase issues - should be fixed now - but the bottom one about incorrect arguments is a problem on master currently.

Yeah, should be. I just re-copied that one file, but let me do a full folder re-copy to make sure it's all what it's supposed to be.


Make the bot multi-language

When i'm finished with #17 and #20 ill take a look if I can pull this off

This will make the bot harder to code and the code is probably going to be a mess. So for now I will close this issue.


replaced size badge with deps and issues

The size badge isn't doing anything useful. Deps and issues are more informative, so I added those instead.
Also, downshift should probably be a dependency, not a peerDependency. The coupling between a set of bindings and the library they bind to is pretty tight, and it seems unlikely that you'll have other dependencies that themselves will depend on downshift.

Thanks for the feedback!
Yeah I’ll probably do that. I also wanted to keep the compatibility to specific versions, meaning that if a new version of the original library comes out that might add new features or introduce breaking changes, the bindings will need to catch up with the new versions.


Remove duplicate Getting Started menu

This address this issue https://github.com/microsoft/azuredatastudio/issues/15980 regarding duplicate Getting Started item.  I'm removing this item contribution since I don't see this contribution in the latest VS Code source code for this file https://github.com/microsoft/vscode/blob/main/src/vs/workbench/contrib/welcome/gettingStarted/browser/gettingStarted.contribution.ts.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.76%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
26232

Relevant Lines:
55689

💛  - Coveralls


Unable to create app

Deploying the app with default avluies on Heroku results in a failure at the Run scripts & scale dynos stage. There is an error message provided: "App creation failed. Please ensure you have valid values in the above form fields."
Can the defaults be updated to deliver the one-click deployment experience expected from a Heroku button?

This might be seen as a duplicate of #2, but they are different. I'm asking here for the button to use default that allow for the intended one-click experience of a Deploy to Heroku button (even if that makes for a limited production app). I'm also finding that #2 affects me, but I feel like it's a different issue, given the workaround posted.

i'm having the same issue

Having the same issue here..


Adding Project Drawing Space

Fixes #361
Screenshots

add screenshots

add screenshots

Done @pranjay-poddar

Hey bro kindly review @pranjay-poddar

Plagiarised Code:
https://github.com/Ayush7614/Bundli-Frontend/blob/main/Drawing App/index.html


Form Restoration and Enhanced Marker Handling

Feature: Form Restoration

Implemented a browser-native "form restoration" mechanism for maps.

Saves map's lat, lng, and zoom levels in an invisible <input>.
Leverages the pageshow event to restore map data when the user navigates back to the page.

Key Benefit: Ensures seamless map state retention without requiring manual reinitialization.

Issue Fixed: Previously, the map's center position was used instead of the marker's position. This update aligns the behavior with user expectations.

Refactor: Simplified State Management

Updated Methods:

setMarkerRange
setFormRestorationState

Improved clarity and reduced complexity for managing marker ranges and form restoration.

New Features: Marker Customization

Added Methods:

markerHtml
markerIconUrl
markerIconSize
markerIconClassName
markerIconAnchor

Purpose: Enable dynamic and customizable marker properties for a more tailored user experience.

Chore:

Updated project documentation to reflect the new features and best practices for form restoration and marker handling.
Enhanced the build script for seamless integration.

Thank you for this contribution!
The changes look clean and well-implemented. I'm merging this now. Great work! 👍


feat(BDropdown): add prop for max height and width for the dropdown menu

Describe the PR
Add max-height and max-width props to the BDropdown component, to allow overriding the size calculated by @floating-ui/vue
Small replication
An example was added to the docs
PR checklist

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix :bug: - fix(...)
[x] Feature - feat(...)
[ ] ARIA accessibility - fix(...)
[x] Documentation update - docs(...)
[ ] Other (please describe)

The PR fulfills these requirements:

[x] Pull request title and all commits follow the Conventional Commits convention or has an override in this pull request body This is very important, as the CHANGELOG is generated from these messages, and determines the next version type. Pull requests that do not follow conventional commits or do not have an override will be denied

I'll note that part of my motivation for this was that I couldn't get the offset to work, but it's probably something in my setup (old app, getting updated). However, from reading the code, it seems like it uses @floating-ui/vue which hard-codes using the CSS top and left, but it feels like for placement: end, it should use right, and so on, no?

What is the use case for this feature? you want to limit the size more than the available space and make dropdown scrollable?
Why couldn't you just use css to make this happen?
Maybe we should but the floating-ui height to css var so it could be used in a class to do the calculation on css side.

I'll note that part of my motivation for this was that I couldn't get the offset to work, but it's probably something in my setup (old app, getting updated).

Can you provide a minimal reproduction of that?

However, from reading the code, it seems like it uses @floating-ui/vue which hard-codes using the CSS top and left, but it feels like for placement: end, it should use right, and so on, no?

no

What is the use case for this feature? you want to limit the size more than the available space and make dropdown scrollable?

Yes, that's it

Why couldn't you just use css to make this happen? Maybe we should but the floating-ui height to css var so it could be used in a class to do the calculation on css side.

I can't (easily) do it in CSS because the max-height gets applied as an in-line style, so it overrides my class (would have to use !important)

Can you provide a minimal reproduction of that?

I'll try, but I'm currently trying to reproduce an error I'm getting with tooltips which isn't reproducing outside the app I'm working on migrating

What is the use case for this feature? you want to limit the size more than the available space and make dropdown scrollable?

Yes, that's it

Why couldn't you just use css to make this happen? Maybe we should but the floating-ui height to css var so it could be used in a class to do the calculation on css side.

I can't (easily) do it in CSS because the max-height gets applied as an in-line style, so it overrides my class (would have to use !important)

so you're saying !important is hard to add to css?
I'm gonna close this since it's easily solvable with css and your code isn't working to begin with. I'll create a issue for myself to convert the height to a variable.


stack overflow running git stack

[X] I have searched the discussions
[X] I have searched the existing issues

Description
I just installed git-stack using cargo install --locked on an up-to-date Arch linux installation.
Then, running git-stack leads to some 40 sec of 100% CPU usage, git-stack then stack overflows.
Version
git-stack 0.2.9
Steps to reproduce
This was on my local clone of kaspar030/RIOT, which has a lot of branches.
I just typed git-stack there.
How can I provide more context?
Actual Behaviour
riot on  master via ⍱
❯ git-stack

thread 'main' has overflowed its stack
fatal runtime error: stack overflow
Aborted (core dumped)
riot on  master via ⍱ took 41s
❯

Expected Behaviour
I don't know the expected output.
Debug Output
https://gist.github.com/kaspar030/b2e524241f7403abc2864181421d94cc

Ouch, hadn't tested on a big repo yet and I can see that happening.  This uses recursion a lot.
As a workaround setting stack.stack to descendants will probably avoid the worse problems.
I might be able to insert some hacks to workaround this but I'm looking at a more fundamental change in our data structure that makes it easier to iterate rather than recurse.


[BuG fix] V3 importer was not detecting engines registered on a file extension

This fixes also this https://github.com/petebrowne/sprockets-sass/pull/40
After some research i found how to detect engines registered on a file extension.
Tested also and works like a charm.
Apparently it seems i no longer have to delete this line     context.metadata.delete(:data) from handle_complex_process_result. Seems to work just fine with that line too.
Let me know what you think. Thanks.

Travis Failed for ruby 2.0 and 2.1.5 because of an issue with bundler on Travis itself, it somehow requires an older version of bundler. I left a comment on this travis-ci/travis-ci#5831 for that, maybe we should use ruby > 2.2.2 for travis .
Tests are passing locally for all ruby versions.

@petebrowne , can you please review this pull request?

Would be nice if we could drop from Travis ruby 2.0 ( since is not maintained anymore ) and maybe 2.1.5 from Travis...they are only causing issues with bundler  on Travis and i don't know any workarounds for them.

That's fine with me

Done. Updated travis.yml file . Let me know if anything needs updating or more tests . Thank you very much

Thank you very much :)

can you also release a new version to rubygems so others can benefit from this bug fix? Thanks.

Released as 2.0.0.beta2

Thanks a lot :)


Invalid route to publish assets?

Hi @mackrais
I installed using:
php composer.phar require --prefer-dist mackrais/yii2-crop-image-section "dev-master"

And configured my form according to the example:
use mackrais\cropimage\ImageCropSection;

But i'm getting this error:

It's trying to create a folder from the root of the app to this route: [app_root]/widgets/MrCropImageSection/assets
Any advice or suggestion?
Thank you very much!

Hi! Are you set in web.php component asset ?
    // Assets js and css
    'assetManager' => [
        'class' => 'yii\web\AssetManager',
        'linkAssets' => true,
        'forceCopy' => YII_ENV_DEV ? true : false,
    ],

I have the same error, I solved it by creating the folder structure @app/widgets/MrCropImageSection/assets and then pasting your css and js folders in there.


Remove unecessary quote

Description:
For historical reason go tags can't have space and the quote escaping was actually done by the shell before. Now, as we call fyne directly, we should not quote tags as once passed to go, it breaks things.
Fixes #195
Checklist:

[x] Lint and formatter run with no errors.
[x] Tests all pass.

Please don't delete the "includes tests" checkbox - it is important that we see 2/3 tasks for the non-optional group so reviewers can decide if tests were not needed.


Audit RNG on platform QPG

On request of Tennessee - check correct RNG functionality on our platform:
Over 2 separate boots of master branch, search for logs with AUDIT: in the line and follow the instructions in PR 10456 to detect if your RNG is properly configured.
Add the logs of 2 different boots on same device to the issue
If everything is OK, immediately close the issue.


Add deploy:revisions_log

Show revisions log of the last 10 deploys.
Did a quick GitHub edit to start the discussion, will add specs and tune stuff if necessary.
cc @masterkain

Do we really need an internal task for cating a file on the server?

It's a cat (tail) on a server of a specific file in a specific path on a remote server to which you connect on some port with an identity file etc. …you got the idea ;)
I need it regularly, especially when pushing branches to staging servers.
Doesn't seem out of place to me as it fits nicely under the definition of automation.
Also writing that file is part of the default deployment procedure and the filename is "owned" by capistrano itself.
That's all I got :)

Closing because it doesn't meet the contribution guidelines.

Does this mean that when will meet the contribution guidelines it will be merged?
I'm asking just to be sure this makes sense before trying to find the time to polish the PR.

It seems like it'll be a lot of work to add tests and everything for this, and I don't just know if it's worth it, it's only a one-liner, plus the boiler plate code, don't feel obliged to work on the PR, but if you can get it tested (which I suspect will be difficult) I'd be happy enough to merge it.


Type hints

Type hints are helpful:

When editing code as many code editors will use type hints for auto completion and indicating problems
Make it easier to understand the code
Can use mypy to statically check whether all types are correctly being used, e.g., for function parameters

There are a range of type hint levels we could adopt:

No type hints
Some type hints, e.g., for key functions and class/ instance variables
Use type hints extensively except where it is impractical to do so

To gain the maximum advantage of type hints and for code correctness to enhance stability of charms, I would recommend we go for (3).

Type hints are helpful:

When editing code as many code editors will use type hints for auto completion and indicating problems
Make it easier to understand the code
Can use mypy to statically check whether all types are correctly being used, e.g., for function parameters

There are a range of type hint levels we could adopt:

No type hints
Some type hints, e.g., for key functions and class/ instance variables
Use type hints extensively except where it is impractical to do so

To gain the maximum advantage of type hints and for code correctness to enhance stability of charms, I would recommend we go for (3).

+1 for option (3)!


Opens in a tiny window

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug
When I start VS Code it opens in a tiny window.  This has been happening for two days now.
VS Code version: Code 1.66.0 (e18005f0f1b33c29e81d732535d8c0e47cafb0b5, 2022-03-30T05:50:14.623Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No
Remote OS version: Linux x64 3.10.0-1160.59.1.el7.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.73GB (7.64GB free)

Process Argv
--folder-uri vscode-remote://ssh-remote%2Bfdvmprdlin1/data/tfs_workspaces/walter.johnson/NXT_DEV/po/PurchaseOrderReceiving/app --crash-reporter-id cf96a43f-1c16-4044-96fc-f782f6b7005f

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: fdvmprdlin1

OS
Linux x64 3.10.0-1160.59.1.el7.x86_64

CPUs
Intel(R) Xeon(R) Gold 5118 CPU @ 2.30GHz (8 x 2294)

Memory (System)
46.99GB (1.39GB free)

VM
100%

Extensions (7)

Extension
Author (truncated)
Version

remote-ssh
ms-
0.78.0

remote-ssh-edit
ms-
0.78.0

remote-wsl
ms-
0.66.0

debugger-for-chrome
msj
4.13.0

vscode-tcw-snippets
Tyl
1.4.2

vscode-eslint
dba
2.2.2

tyler-genero
Mun
0.0.4

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vsclayoutctrt:30451275
pythonvsnew555:30457759
vscscmwlcmt:30438805
vscgsvid2:30447481
pynewfile477:30463512

Can you still reproduce this issue with the latest release (1.68)

Yes, this is still happening.
From: SteVen Batten @.>
Sent: Thursday, June 9, 2022 5:28 PM
To: microsoft/vscode @.>
Cc: Johnson, Walter @.>; Author @.>
Subject: Re: [microsoft/vscode] Opens in a tiny window (Issue #146540)
Can you still reproduce this issue with the latest release (1.68)
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/microsoft/vscode/issues/146540*issuecomment-1151643224__;Iw!!A69Ausm6DtA!bJ7C1dQByZE_c8hf5tPa6xtJxOiFo6o06N_-C7hRT0JSWR6L3f0IfZNwr7V8VqC8HECbZwvuYriKuyKJ3m5czj5rIB2Fr4U$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AGKZO45SHOJZT6IKBUJV27LVOJOXZANCNFSM5SIQJ43Q__;!!A69Ausm6DtA!bJ7C1dQByZE_c8hf5tPa6xtJxOiFo6o06N_-C7hRT0JSWR6L3f0IfZNwr7V8VqC8HECbZwvuYriKuyKJ3m5czj5rCeWhVj0$.
You are receiving this because you authored the thread.Message ID: @.***>

Does this happen when you only have 1 window to restore? or if you have many windows?

This is how it opens up:

And I always have to resize it to make it large enough to be useful.

@w3johnson do you have multiple monitors? if so can you share info about the resolution and scaling of each monitor in your OS settings?
@deepak1556 I notice if I restart VS Code on my second monitor (which is 2K while my primary monitor is 4k) the window gets smaller each successive time.

Yes, my laptop is:

and the other monitor (where I open VS Code) is:

Hey @deepak1556 @sbatten, this issue might need further attention.
@w3johnson, you can help us out by closing this issue if the problem no longer exists, or adding more information.

It still happens every time. I always have to manually resize the VS Code window to make it useable.

This issue seems to have been fixed with the latest version (1.71.0) of VS Code.


X.A.Minimize: Add withMinimized

Did you also want to export withMinimized?

Did you also mean to export withMinimized?

yeah... oops


upd(qtile-git): fix installation error

Fixed installation error (unable to verify version from pyproject.toml) for qtile-git.
Updated dependencies.

I wanted to ask, for the qtile-extras package, qtile is a dependency, so I should just say pacdeps=("qtile-git") in the qtile-extras pacscript, right?

Yes.

Ok, thanks


order readme correction

Doc reference : https://razorpay.com/docs/api/orders/#create-an-order

@sofiya02 in create order "partial_payment" param is default "false" if we dont pass in it


fn CLI update doesn't support annotation values that aren't JSON blobs

According to https://github.com/fnproject/docs/blob/master/fn/develop/annotations.md it should be possible to specify an annotation value which is just a plain string. Currently, the CLI does not allow this, instead requiring that the value must be a parseable JSON string.
For example:
fn update function myapp myfn --annotation someannotation="foo"

fails with Unable to parse annotation value 'foo'. Annotations values must be valid JSON strings.
Performing the same operation using curl against the fn REST API succeeds however.

Turns out this is not an issue with the CLI but rather with the shell stripping out the quotes. Adapting the above example to:
fn update function myapp myfn --annotation someannotation='"foo"'

provides the right behavior.


每阶段大概的训练时间是多长呢

如标题啦

stage1 takes approximately 9.6 hours,
stage2 takes about 64.8 hours, and stage3 takes 29.4 hours.


[Feature Request] Add a meta field that lists all the created atlas

When multiple output files get generated due to having too many images in your atlas there is no way of knowing how many of them were created.
Maybe a file that tells you how many atlas were created could be useful?
(I would make this an option to always do it (even if only one atlas was created) so from code I can just tell my loader to check one file and load all the things that said file knows: be it one or many atlas.)
anyway, awesome work!
Edit: Maybe a metadata field for that would be simpler and break less stuff?
Edit2: the metafield related_multi_packs could host an array of strings naming all the other files created by the packer. We could inject it into the options object here: https://github.com/odrick/free-tex-packer-core/blob/master/FilesProcessor.js#L27 however I am not sure how this would affect the templating system :S
(It seems that the metafield related_multi_packs is what is used for that in other packers)

Merged


News Added and issue Pet News Addition #977

Issue number #977 resolved , please check and merge and give proper labels of gsoc and levels

@akshitagupta15june Please check

Add a screenshot of pet news page

Add a screenshot of pet news page

Screenshot added


docker-compose up fails with missing file

running docker-compose up fails with the following error:
Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "/app/child-validator.sh": stat /app/child-validator.sh: no such file or directory: unknown

The file child-validator.sh doesn't exist in the entrypoints dir (nor does it exist in the image itself):
$ docker compose run -it --entrypoint /bin/ls validator1 -la /app
total 24
drwxr-xr-x    1 root     root          4096 Nov 16 01:07 .
drwxr-xr-x    1 root     root          4096 Nov 16 11:19 ..
-rwxr-xr-x    1 root     root           831 Nov 16 01:06 bootstrap.sh
drwxr-xr-x    9 root     root          4096 Nov 15 22:57 fendermint
-rwxr-xr-x    1 root     root           148 Nov 16 01:06 fendermint.sh
-rwxr-xr-x    1 root     root           646 Nov 16 01:06 validator.sh

Yeah, sorry, this was missed, the script is actually called validator.sh it was fixed in the latest commit (that also addressed your other issue!)
Thanks!

closed.


Instantiate on scroll listener on the document rather than one for every pop

And yes... I know I spelled listener wrong in the PR..

LGTM other than that one comment!


Msprivate/master

Included is my current working version of libcanard. canardReadRxTransferPayload is the only missing function right now. Recent updates to the design document have also not been implemented (ftp://pkir.net/libcanard.diff). canard_test.c, based on simple_sensor_node is what I've been using to test the library. It sends 3 data types on the can interface. There's also a function, random_drop(x), called on the bottom of the send thread that will randomly drop a frame. Set that to the probability you want frames dropped, 0 if you dont want anything dropped. Right now canard_test just prints out the data received from the library.
To do:
implement crc checking
ignore incomplete transfers (probably has to do with the above)
ask any questions!

Wow impressive PR! I will try to review it tomorrow and fix the build (#2)
so we can check if it builds.

Giant leap for the project! Thanks Michael.
I added a few comments, and probably @antoinealb will add some more later.
You can use pyuavcan to hunt bugs in the transport layer implementation. Just checkout the repo and run the following script in its directory:
#!/usr/bin/env python3
import logging, sys
logging.basicConfig(stream=sys.stderr, level=logging.DEBUG)
import uavcan
node = uavcan.make_node('vcan0')
while True:
    try:
        node.spin()
    except Exception:
        logging.error('UAVCAN error', exc_info=True)

It will report any anomalies on the bus to stderr in more or less human-readable form.

I added a few specific comments to the file, but @pavel-kirienko knows more than me about the internals of the protocol.
Additional points:

I would prefer if the build system was CMake instead of a makefile as it makes it more portable. I can implement it after the merge though.
The example node should go into another folde e.g. examples/.
Some licensing informations are wrong. Is it an automatic placeholder or do you have a problem with the current licensing ?

I tested it on an Ubuntu machine and it seems to work. Very nice :+1:
Thanks again for this big contribution!

The code can be autoformatted to match the style guide with uncrustify. See https://github.com/Zubax/zubax_style_guide#autoformatting.

I would prefer if the build system was CMake instead of a makefile as it makes it more portable. I can implement it after the merge though.
The example node should go into another folder e.g. examples/.

The makefile only builds a demo app. My understanding is that the demo application is only needed for testing/debugging (instead of unit tests), and it will be removed once the development is finished. @masierra is the assumption correct? Actually, it would be nice to convert the testing app into unit tests, but that will require some effort. Anyone willing to undertake this conversion?
As for the build system for the library (weird that we haven't discussed it earlier), we need to evaluate the intended usage scenarios. The whole library is just one trivial C file and one API header, which allows to build it directly with the application - no separate build system needed:
Make-based application:
SRC += libcanard/src/canard.c
INC += libcanard/src

CMake-based application:
include_directories(libcanard/src)
add_executable(... ... libcanard/src/canard.c)

Actually, looking at the above I'm beginning to suspect that it might be beneficial to move everything from src/ to the root directory, for the sake of simplifying inclusion. We're not going to add other .c or .h files anyway (since the library is one-file by design). @antoinealb what do you think?

We can move everything in the root directory yeah. (maybe keep separate dirs for examples and unit tests).
The included demo is a full blown example that exchanges messages and publishes node status. I think it is good to keep it that way to help people starting a project with libcanard.

We can move everything in the root directory yeah. (maybe keep separate dirs for examples and unit tests).

Yes, that's what I meant.

The included demo is a full blown example that exchanges messages and publishes node status. I think it is good to keep it that way to help people starting a project with libcanard.

OK, let's keep it then (it could use some prettification). We'll need to add some tutorials to http://uavcan.org/Implementations/Libcanard anyway though.

The latest changes look good.

LGTM too :+1:

I think its ready for you guys to take a look!

I suggest to uncrustify the code, see https://github.com/Zubax/zubax_style_guide#autoformatting

We're still missing canardReadRxTransferPayload().

@masierra Could you please merge master into your branch so we could get CI working? Thanks.

@pavel-kirienko No need to, you just have to restart the build. Travis merges the PR in master before trying to build.Source: https://docs.travis-ci.com/user/pull-requests
Apparently the build is currently legitimately failing (see logs for details).

Fun! Seems like it's failing here: i<CANARD_RX_PAYLOAD_HEAD_SIZE, where i is uint8_t (which is also a mistake it should be uint16_t as well as data_index). This is likely because Travis is trying a 64-bit build. 64-bit platforms are not supported by design.
I guess we need to add -m32 here https://github.com/UAVCAN/libcanard/blob/master/.travis.yml#L13

Let me elaborate: on 64-bit platforms, due to large pointer size, the value of CANARD_RX_PAYLOAD_HEAD_SIZE may be zero or even negative. In order for the code to work it must be strictly positive.

Also while we're at it, can we add strict flags like -Wall -Wextra -pedantic?

Coverity detected some issues, mostly with the test application: https://scan.coverity.com/projects/uavcan-libcanard/view_defects

Need access :(

Sent you an invite.
On Jan 30, 2016 2:35 AM, "Michael Sierra"<EMAIL_ADDRESS>wrote:

Need access :(
—
Reply to this email directly or view it on GitHub
https://github.com/UAVCAN/libcanard/pull/3#issuecomment-177018237.

Sorry about my silence, I just noticed the new commits. Github doesn't notify about updates to PR.
@masierra do you feel this is ready now? @antoinealb do you?
Also, Michael, I sent you an invite to the UAVCAN team a while ago so you could work in the original repo without having to maintain a fork.

Hmm I don't think I saw that. But yeah, I think it's at a good point right now and then we can fix issues down the line as we find them.

I just re-sent the invitation, let me know if it didn't work. I am merging this huge PR now, so we can continue to work on the library in smaller increments.

looks good to me. Works on Ubuntu 14.04 via virtual can interfaces. I didn't have a chance to test it "for real" though.


Build and safeguard remaining routes

Title: Build and safeguard remaining routes
User Stories

As a logged-in user, I want to be able to create new posts and comments and upvote other posts

fixed with commit 73707c1


Oauth - Proposal

As an engineer, it would be cool to build applications that c0d3 students could interact with.
These applications are not core to c0d3.com, but could be experimental / supplemental to c0d3.com.
In the following examples, Oauth at c0d3.com would be powering authentication so students don't have to create an account for each of the sites:

For example: (HR review)
Students have trouble drawing from their experience to answer HR questions. Imagine a HR application where:

Students go to hr.amayz.dev
Students click on (connect to c0d3) to login, so they don't have to create a whole new login for hr.amayz.dev
Students could now click on HR questions and submit their answers and get feedback about it from other students.

For example: (Flashcards)

students go to flashcards.student2.dev
Students click on (connect to c0d3) to login, so they don't have to create a whole new login for flashcards.student2.dev
Students could now work on their flashcards and save their progress.

For example: (Daily Updates)

students go to daily.student3.dev
Students click on (connect to c0d3) to login, so they don't have to create a whole new login for daily.student3.dev
Students could now post daily updates on their progress through c0d3.com. Maybe start a sprint team with other students.

For example: (Interview Solutions)

students go to interview.student4.dev
Students click on (connect to c0d3) to login, so they don't have to create a whole new login for interview.student4.dev
If students have not passed JS3, they would be denied access. Otherwise, Students could now submit the questions they received in their interviews and get "pro" solutions (solutions written by our engineering team).

Etc. This also opens up an opportunity for students to create their own applications to share with other students.

Super exciting!


Add el2g integration documentation

Signed-off-by: Andy Doan<EMAIL_ADDRESS>Readiness

[ ] Merge (pending reviews)
[ ] Merge after date or event
[x] Draft

Overview
We are delivering a new feature in our next LmP release. We still need to work out some of the cross-company documentation and wording, but I'd like to get some early feedback on the actual instructions I've got. Could you please help me clean this up some so that I can share it?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[ ] Run spelling and grammar check, preferably with linter.
[x] Avoid changing any header associated with a link/reference.
[x] Step through instructions (or ask someone to do so).
[ ] Review for wordiness
[ ] Match tone and style of page/section.
[x] Run make linkcheck.
[x] View HTML in a browser to check rendering.
[x] Use semantic newlines.
[ ] follow best practices for commits.

[ ] Descriptive title written in the imperative.
[ ] Include brief overview of QA steps taken.
[ ] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[ ] Squash commits if needed.

[ ] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for a9ea1aa0c8fa722a952c4922dd35d3281af45050 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1509/docs/artifacts/html/index.html

Hi @doanac, I started my initial read through,  and will leave some comments. I will finish up on Thursday. I'll keep reviewing it as we progress.

Docs for 090a4ad1cefe79306a03ce39503bd252fdebef02 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1514/docs/artifacts/html/index.html

Docs for 8d78ed7242c1e92df38874eb993aeb4062d950ae are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1515/docs/artifacts/html/index.html

Build failure my fault, did not extend the underline when I suggested a change to a section header.

Docs for 34895b627c64bf697b91ccf3f373e7876c0588e2 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1519/docs/artifacts/html/index.html

Docs for b9180b52497383eb469ee26e5db09eb7665262b0 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1520/docs/artifacts/html/index.html

Docs for 2f9f02c86aa0808a5086adfb0cd8cb4d056ff6af are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1521/docs/artifacts/html/index.html

Docs for a2ab608f68516331f5948123c6726f8ff1a0c81d are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1522/docs/artifacts/html/index.html

Docs for f504c2a3ae4bb44dc48e255b8c5bf222d8a70b69 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1523/docs/artifacts/html/index.html

Docs for 7f399b4c352c86fce9422a9a287587183c35ed5d are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1524/docs/artifacts/html/index.html

Docs for 0bbf8ef76dbde9b0cc1fa28337d587edc95b46fc are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1525/docs/artifacts/html/index.html

Docs for 8b1069a19a391207f9f00ab5f26ccfff36ea73dc are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1526/docs/artifacts/html/index.html

Docs for a7e09c11bebb1e98f681b21159c47e9e7b8d5291 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1527/docs/artifacts/html/index.html

Docs for f2c5f1fea453bd3b4a406af34d9f7331c1724e4b are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1528/docs/artifacts/html/index.html

Docs for a1645ae025d0141cd20145eff89c6fb4e8ae441c are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1529/docs/artifacts/html/index.html

Docs for c990289d0a84ec8b5310f32a87a24a0e49838864 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1530/docs/artifacts/html/index.html

Docs for c2a36d81896af8bbf7aab0cc4f4bfcedb6627828 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1531/docs/artifacts/html/index.html

Docs for 5d5725c701f75e8b5f5cf251bb749882aa71d11b are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1532/docs/artifacts/html/index.html

Docs for ee6954f730acee3255c8e4418b3441145fb136dd are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1533/docs/artifacts/html/index.html

Docs for ee6954f730acee3255c8e4418b3441145fb136dd are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1534/docs/artifacts/html/index.html

Docs for ee6954f730acee3255c8e4418b3441145fb136dd are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1535/docs/artifacts/html/index.html

Docs for b7c9d9147f87cfbc2cd964332c07ad72826791cf are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1536/docs/artifacts/html/index.html

Docs for 875ce8b16b6085f28fafd3872c8c1b5144769958 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1537/docs/artifacts/html/index.html

Docs for e4c2fb4971232477ff6418387d1dc8617944fe22 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1539/docs/artifacts/html/index.html

Docs for 97fc42ded5557a02d1be0a4841c2f959ddded2f1 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1540/docs/artifacts/html/index.html

Docs for eadcaa9864a1889c3af8c406eee0b9962eb72183 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1541/docs/artifacts/html/index.html

Docs for 9a9448b97d2c0749a210190d19286f2b9a1668b3 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1542/docs/artifacts/html/index.html

Docs for 51ccd5ae759307fa163deb816c1139f1613413f5 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1574/docs/artifacts/html/index.html

squashed and re-based on main.

Docs for bb25442dd36f446ea39552beed63cd0c94723aed are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1575/docs/artifacts/html/index.html

@doanac are we ready merge? (It's still marked as a draft, so I will hold off until you give the go)

@kprosise - not quite. hopefully soon

Docs for b3a9b5b32ceb31e3f53b9ca8d7ab93760ecae51c are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1577/docs/artifacts/html/index.html

Docs for bfadcb18abf79a546c1a25814c3c3eca50867655 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1578/docs/artifacts/html/index.html

Docs for 169312f818aa5dbe99d0e904611bd9ea3d8c26e8 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1642/docs/artifacts/html/index.html

Docs for c5e1bcbf25af2408b0e86172716bb6a06d856cba are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1643/docs/artifacts/html/index.html

Docs for ff718999c0704c2afbed24a99364a3e4b48d96f3 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1644/docs/artifacts/html/index.html

Docs for cee91285c574087facd474021ef3ce8068122263 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1645/docs/artifacts/html/index.html

Docs for 3ad2246cce0eb62456e13e6e1f79873a2580ed88 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1646/docs/artifacts/html/index.html

Docs for b6f1ed561596c7f8dc45b4c769c393b2922589f4 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1647/docs/artifacts/html/index.html


repo: mandatory issue templates (AIDM-428)

Description

adds a bug report and feature request issue template
disables blank issue creation
adds a security document

Motivation

standardizes the create issue experience across tracers
see the node.js create issue screen for an example of what this looks like

Additional Notes

does the C++ tracer instrument third party packages? I'm assuming not and have left such fields out
let me know if there's any other things you need to know (musl, glibc, etc) and we can add fields

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.91%. Comparing base (febab81) to head (c959474).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #175   +/-   ##
=======================================
  Coverage   93.91%   93.91%
=======================================
  Files          73       73
  Lines        4176     4176
=======================================
  Hits         3922     3922
  Misses        254      254

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


[Feature][Dynamic] Dynamic-created Subtasks Fail to Run Using Tenant Code Specified at Task Start

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
I've encountered an issue with Dynamic tasks in DolphinScheduler where the tenantCode specified at the start of the task is not being passed to the subtasks. This results in the tenantCode being null when constructing the command for the day's task.
Upon reviewing the source code, I found that the createCommand method in DynamicCommandUtils does not transfer the ProcessInstance's tenantCode to the command object. This omission seems to be the cause of the issue described above.
Use case

Create a sub-workflow that includes a shell script task to output the current running role:

current_user=$(whoami)
echo "whoami: $current_user"

Create a parent workflow that includes a Dynamic task pointing to the sub-workflow, and associate it with a script task to output the parent task's role:

current_user=$(whoami)
echo "parent task whoami: $current_user"

Run the parent workflow and set the tenant (tenantCode) to root.
Observe the log outputs.

Expected Behavior:
The log output of the sub-workflow task instances should show the role as root.
Actual Behavior:
The log output of the sub-workflow task instances shows the role as default, indicating that it was not passed from the parent task.

Possible Solution:
It may be necessary to modify the createCommand method in DynamicCommandUtils to include the ProcessInstance's tenantCode when constructing the command object.
I believe this adjustment will ensure that the tenantCode is correctly passed to all subtasks, maintaining consistency and the appropriate execution context.
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Similarly, the listParameters type parameters in Dynamic nodes do not accept parameters with the same name passed at the start of the task. Theoretically, it's not always possible to know how many sub-nodes need to be split when creating the task. It might be necessary to pass this information during task execution.

This is more like a bug.


pro/con vs. timescaledb

cool project.  I'd be interested in anything reasons why this might be better than using timescaledb.

The main advantage is it works on a stock PostgreSQL installation, e.g. on AWS RDS. There's no need to install extensions.


VMaskTextField  addTarget forControlEvents:UIControlEventEditingChanged

@viniciusmo , any ideas, why textField add target not working!
VMaskTextField *textField
[textField addTarget:self action:@selector(textFieldDidChange:) forControlEvents:UIControlEventEditingChanged];

-(void)textFieldDidChange :(UITextField *)theTextField{
    NSLog( @"text changed: %@", theTextField.text);
}

It's a problem in [VMaskEditor shouldChangeCharactersInRange:replacementString:textField:mask]. In that method the textField.text value is set manually on code. When this happens the UIControlEventEditingChanged event is not trigered.
A solution is to paste the following line of code just next to the places where the textField is set in that method:
textField.text = returnText;
[textField sendActionsForControlEvents:UIControlEventEditingChanged];

@felipeferri Good!, already fix that issue


Tabs: Adding Dynamic Tabs causes blank tabs

Info
Given this code:
const [isReport, setIsReport] = useState(false);

function onClick() {
  setIsReport(true);
}

<GoATabs>
  <GoATab heading="1">Content 1</GoATab>
  <GoATab heading="2">Content 2</GoATab>
  <GoATab heading="3">Content 3</GoATab>
  {isReport ? <GoATab heading="Report">Some Random Content</GoATab> : null}
</GoATabs>

<GoAButton onClick={onClick} mt="3xl">
   Add
</GoAButton>

You get this:

And then when you click the button, you get this:

This worked in the previous version, hence the critical nature of this bug.
Acceptance Criteria:

Adding Dynamic Tabs should work without adding blank tabs to the beginning

Closing as this has been released, no idea why it's still open


[ownership] Change CanonicalOSSALifetime to use an InstModCallback.

This enables passes to use this as a utility that properly composes with how the
pass maintains its state. If an InstModCallback isn't passed in, we use the
default InstModCallback which should be cheap (always succeeding check for
nullptr + call inline default callback).

Should be NFC

@swift-ci test

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - 953bfb29549e60110a18c9cf48f51e0c33136718

Build failed
Swift Test Linux Platform
Git Sha - 953bfb29549e60110a18c9cf48f51e0c33136718

@swift-ci test


Relational query returns 0 object on iOS SDK

Make sure these boxes are checked before submitting your issue -- thanks for reporting issues back to Parse Server!

[X] You've met the prerequisites.

[X] You're running the latest version of Parse Server.

[X] You've searched through existing issues. Chances are that your issue has been reported or resolved before.

Environment Setup
Ubuntu 14.04 64 bit
Parse-Server 2.1.6
Steps to reproduce
I'm doing relational query to get opening hours of a restaurant where restaurant name matches certain keywords. When I'm querying it on my iOS app, it returns 0 object.
PFQuery *restaurantQuery = [PFQuery queryWithClassName:@"Restaurant"];
	[restaurantQuery whereKey:@"name" equalTo:self.selectedRestaurantName];

	PFQuery *hoursQuery = [PFQuery queryWithClassName:@"Hour"];
	[hoursQuery whereKey:@"restaurant" matchesKey:@"objectId" inQuery:restaurantQuery];
	[hoursQuery whereKey:@"days" equalTo:[NSString stringWithFormat:@"%ld", (long)[Utility getWeekday]]];

	[hoursQuery findObjectsInBackgroundWithBlock:^(NSArray * _Nullable objects, NSError * _Nullable error) {
		if (!error) {
			self.restaurantHours = [[NSArray alloc] initWithArray:objects];
		} else {
			NSLog(@"%@", error);
			self.restaurantHours = [[NSArray alloc] init];
		}

		[self.tableView reloadData];
	}];

But when I tried to query it using cURL, it returns the result that I needed.
curl -X GET -H "X-Parse-Application-Id: myAppID" -H "X-Parse-REST-API-Key: myRESTKey" -G --data-urlencode 'where={"restaurant":{"$inQuery":{"where":{"name":"selectedRestaurantName"},"className":"Restaurant"}},"days":"3"}' https://my_url.com:1337/parse/classes/myClass

How did you migrate your data? We had this issue when we exported data from Parse and then imported into Mlab.com. The problem with that is that it formats the objects slightly differently. Not sure if you're running into the same issue here but maybe check the data migration?

@harrygt how different is the format? How did you solve your issue?

Basically when we imported into mLab.com using this:
mongoimport -h databaseURL -d databaseName -c PDF -u user -p password --jsonArray PDF.json
We would get "parentInspection" as the name of the relationship to this PDF's parent Inspection. But in iOS this didn't work and would return 0 results. (it also wasn't clear we had to use --jsonArray, before using that mlab.com would just import everything into one document instead of individual objects).
So we then just went ahead and used the Parse Migration button and let that complete. Now the parentInspection relationship is called '_p_parentInspection' and iOS works fully.
So not sure why the p is not added when importing using mongoimport but it is added when using the migration button on parse.com.

@harrygt so what you did was getting all the collections from parse through the export button, then importing the results with mongoimport right?

Yep exactly. See the example I just added to previous comment.

Thanks! We have other issues related to that (like #1014) and you don't notice any error with relationalQueries like matchesQuery etc...?

@harrygt I migrated my data using Parse Migration button.

I'll close that one as it's a duplicate from #1014

We seem to have no problems but if the file isn't found in a getDataInBackgroundWithBlock then we do get [Error]: Response status code was unacceptable: 404 (Code: 1, Version: 1.12.0). We are using AWS with S3Adaptor so not sure if that has anything to do with it but we only get that error code on files that were not uploaded to S3 during our testing.

@harrygt the thing is I don't get any errors at all so I'm not sure what went wrong.

Maybe try:
[hoursQuery<EMAIL_ADDRESS>equalTo:self.selectedRestaurantName];
I'm sure you could do that which avoids the matching query? Other than that I am not entirely sure sorry!

@harrygt still not working but thanks a lot for your help!

What is restaurant? an pointer or just the objectId?

Sorry I couldn't solve it!
Maybe take a look at the data in mlab.com and see which way its formatted. Maybe that will help figure it out like this example:
mongoimport way:
"parentInspection": { "__type": "Pointer", "className": "Inspection", "objectId": "PNLEniZswJ" }
parse.com migration:
"_p_parentInspection": "Inspection$yGts7DGTry"

@flovilmart restaurant is a pointer to an object in Restaurant class

if that's like that then :+1:
[hoursQuery whereKey:@"restaurant" matchesKey:@"objectId" inQuery:restaurantQuery];
// should be

[hoursQuery whereKey:@"restaurant" matchesQuery:restaurantQuery];

@flovilmart wow that solved it! Thanks a lot!


[docs] Provide more documentation on custom executors with global actors

Description: It could be confusing to adopters who were led to believe by the types that they should "just" implement the sharedUnownedExecutor property, but insead they have to implement the unownedExecutor on the specific actor type.
Adding documentation clarify this as well as a simple test that exercises this explicitly; We seem to have much coverage of main actor, but not so much of custom executor global actors.
Original PR:
Scope/Impact: Docs only, explain what should and should not be implemented to make a global actor use custom executors.
Risk: Low, docs only.
Testing: CI testing, added a test to verify this specific pattern
Reviewed by: @DougGregor @hborla
Original PR: https://github.com/apple/swift/pull/74319
Radar: rdar://129647025

@swift-ci please test

@swift-ci please test


Use of pyOpenSSL (upgrade to cryptography)

Hi,
I've just started playing around with django-x509, and it does exactly what is says on the tin; thank you very much for providing this library!
I'd like to be able to specify multiple OUs for certificates, however this does not appear to be supported by pyOpenSSL. In addition, pyOpenSSL has been deprecated for some time by the maintainers and recommends that users migrate over to the cryptography library, which incidentally does support multiple OUs.

(Before anybody suggests it, I'm not part of any hacktoberfest malarkey, I'm an actual user of this library!)

@davidjmemmett thanks for reporting this, we are not aware of the deprecation, can you point out in which communication was this information published by pyOpenSSL?
As long as we maintain backward compatibility I don't see a problem in migrating to the cryptography lib if that's the way to go for the future.

Hi @nemesisdesign,
As per the opening paragraph on pyOpenSSL's README, and according to the history of said file, it's been deprecated since ~16th May 2018.

I think PRs are welcome on this. Sooner or later we'll have to upgrade.


A new guide for how to send email from Spring Boot.

@gregturn Please give this a review when you can. Thanks. :)


Is this project still alive (in development mode)?

Hi,
I wanted to say that this is an amazing project! beautifully crafted.
I wanted to ask if this project is still in development now that Babylon Native is "almost  good for production"?

Hi,
Thanks for the nice feedback!!
Keeping up with all the changes for Babylon.js v5.0 turned out to be too much effort for 1 person and the quality of this project was getting worse due to all the code changes. For these reasons, I stopped the development and the code is now back on an older stable Babylon.js v4.2.0 version.
Indeed, Babylon Native has become much more stable over the years, has many contributors and is officially supported and actively developed by the Babylon.js team, so I would recommend Babylon Native for native application development.


[SDP-1437] main.sh script kills all containers including non-sdp containers

What

Script was deleting all containers.
Change main.sh script to only kill sdp-mtn containers.

Why
People love their containers and don't want them dead.
Checklist
PR Structure

[x] This PR has a reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR title and description are clear enough for anyone to review it.
[x] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).

Thoroughness

[x] This PR adds tests for the new functionality or fixes.
[x] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[x] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[x] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[x] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[x] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[x] This is not a breaking change.
[x] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr480.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr480.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr480.previews.kube001.services.stellar-ops.com


Manually implement IAccessible

IAccessible is a very strange interface. It defines property getters and setters that take arguments, which are represented in C# by get_* and set_* methods. The source generator cannot handle this syntax correctly, even if modified to recognize property get/set methods with arguments. With this change applied, the test app builds and runs without any problems after NativeAOT. I have not verified that the interface properly provides automation information to the system, because I do not know how to do so.

@kant2002 Once this and #10 are merged, could you do me a solid and cut a new release to NuGet.org? I would like to use this library in my personal projects now that the bugs I have found have been squashed. Thanks!

I would like to have VARIANT marshalling implemented differently and preferably in codegen. As you notice I did not implement get/set pattern properly, so that's another problem which I have plans to implement.
I can cut release with these changes, but after I introduce these changes you may need to re-test application.
If you tell me a bit more about scenario where you need IAccessibility, I'm gladly test on them to be a bit more pro-active with potential problems.

Also important detail for me, do you target .NET 5.0 or .NET 6.0 (if so, is it preview or nighlty?)

@kant2002 Honestly, I implemented IAccessible simply because I thought I could. If you would prefer to close this PR, no skin off my back. (Also, what would you want to do differently about VARIANT marshaling? The technique I chose seems to be the simplest way to do so.) Finally, I target both .NET 5 and .NET 6 (and I use only the official preview builds of the latter). I am trying to migrate all my projects from .NET 5 to .NET 6, but it’s a slow and gradual process. Thanks!

I would start with creation of test in similar spirit as StringParameter. https://github.com/kant2002/WinFormsComInterop/blob/fdf3958e7e418f6b0d88d81b3446efcfde95c376/WinFormsComInterop.SourceGenerator.Tests/RuntimeCallableWrapperTest.cs#L1072
with difference approximately like that
System.Variant local_1;
Marshal.GetNativeVariantForObject(dwOrigin, (System.IntPtr)&local_1);
result = ((delegate* unmanaged<System.IntPtr, long, System.IntPtr, ulong*, int>)vtbl[3])(thisPtr, dlibMove, &local_1, plibNewPosition);

System.Variant is most likely for me temporary solution, because I definitely need class which is not so tightly coupled with built-in COM in .NET and Variant marshalling required that I codegen support library in the project, since built-in VARIANT marshalling a bit tricky. I would again consult with previous implementation which I done here https://github.com/dotnet/runtimelab/pull/1142/files and try to get as much as possible from there. On introspection, maybe System.Variant would be good.
Then I have to declare class which implement that kind of marshalling. VariantMarshaller let's say. I would start with empty derived class from Marshaller and add methods as nescessary.
Look for https://github.com/dotnet/runtimelab/blob/feature/DllImportGenerator/designs/Pipeline.md#stub-code-generation for approximate design of marshalling. I use different naming, but thinks that's most complete way how marshalling structured.
I want split net5.0 and net6.0 targeting of WinFormsComInterop since I can limit amount of supported interfaces, System.Drawing now works without my library in .NET 6 Preview 7. I can postpone that split for now.

@wjk in addition to previous comment. If you don't need your changes directly in your apps, I can release as is, so you can use latest verison.

@kant2002 I am OK with releasing both NuGet packages as-is, without the changes in this PR. As I said, this was a very low-priority change. Thanks!


System.MissingMethodException

After investigation for few hours i finally found the reason of that exception.
This package is not compatible with refit version greater than 4.7.9 even if the Nuget dependencies doesn't specify it.

We also encountered this same problem. When using Refit version 4.7.9 in our project, the Camunda client worked fine. However, after upgrading to Refit version 5.0.33, we started seeing these "System.MissingMethodException" errors.
Here is the stack-trace:

System.MissingMethodException : .ctor
at System.Reflection.CustomAttribute.FilterCustomAttributeRecord(CustomAttributeRecord caRecord, MetadataImport scope, Assembly& lastAptcaOkAssembly, RuntimeModule decoratedModule, MetadataToken decoratedToken, RuntimeType attributeFilterType, Boolean mustBeInheritable, Object[] attributes, IList derivedAttributes, RuntimeType& attributeType, IRuntimeMethodInfo& ctor, Boolean& ctorHasParameters, Boolean& isVarArg)
at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeModule decoratedModule, Int32 decoratedMetadataToken, Int32 pcaCount, RuntimeType attributeFilterType, Boolean mustBeInheritable, IList derivedAttributes)
at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeMethodInfo method, RuntimeType caType, Boolean inherit)
at Refit.RequestBuilderImplementation.AddInterfaceHttpMethods(Type interfaceType, Dictionary2 methods) in d:\a\1\s\Refit\RequestBuilderImplementation.cs:line 65   at Refit.RequestBuilderImplementation..ctor(Type refitInterfaceType, RefitSettings refitSettings) in d:\a\1\s\Refit\RequestBuilderImplementation.cs:line 55   at Refit.RequestBuilderFactory.Create[T](RefitSettings settings) in d:\a\1\s\Refit\RequestBuilderFactory.cs:line 18   at Refit.RestService.For[T](HttpClient client, RefitSettings settings) in d:\a\1\s\Refit\RestService.cs:line 28   at System.Lazy1.ViaFactory(LazyThreadSafetyMode mode)
at System.Lazy1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)   at System.Lazy1.CreateValue()
at Camunda.Api.Client.CamundaClient.get_Deployments()
... (our code here)

We were attempting to invoke this function:

var deploymentInfo = await camunda.Deployments.Create($"{id}", true, false, "Workflow API", tenantId, new ResourceDataContent(content, $"{id}.bpmn"));

When we stepped through this on a debugger inside CamundaClient.cs, we see this code:

public DeploymentService Deployments
{
get
{
return new DeploymentService(this._deploymentApi.Value);
}
}

In our case, the "_deploymentApi" was null.  We have not gotten a chance to debug any further.
To reproduce this issue:

Create a new project
Add the Refit Nuget package 5.0.33
Add the Camunda API client
Attempt to execute the camunda.Deployments.Create function.

The problems here start even earlier - when you try to build Camunda.Api.Client with Refit 5.x it fails for the CustomerUrlParameterFormatter. So, yes: Camunda.Api.Client as is is not compatible with Refit 5.x
Upgrading Camunda.Api.Client to Refit 5.x proves to be a bit of a challenge - as soon as I start to do that I get a ton of CS0246 errors with types/namespaces all of a sudden not known to net461. To mitigate that a little faster I tried to set the target framework to netstandard2.0
With that I get about the only breaking change of Refit that affects us directly. In CamundaClient.cs there is a CustomUrlParameterFormatter class (deriving from Refit.DefaultUrlParameterFormatter) with a now invalid method signature on Format.
Refactoring it to
`            public override string Format(object parameterValue, ICustomAttributeProvider attributeProvider, Type type)
{
if (parameterValue is bool)
return string.Format(CultureInfo.InvariantCulture, "{0}", parameterValue).ToLowerInvariant();
            return base.Format(parameterValue, attributeProvider, type);
        }

`
at least makes it compilable, following up with two warnings on Newtonsoft related stuff (gets automatically upgraded to 12.0.3) in StringEnumConverter.cs and CamundaClient.cs
You can find the changed code here:
https://github.com/gaensebluemchenritter/Camunda.Api.Client/commits/master
No guarantee it will work, right now I'm out of time ... and also I'm not entirely sure, why net461 is not working after applying the upgrade to Refit 5.0.23 (as it also defines a net461 target framework in its project file).

At a first glance (using our own project) it seems to work - please give it a try and feedback! Thanks!

Solved the aforementioned problem with net461, so the changed code is compatible with both target frameworks again.
At a first glance (using our own project) it seems to work - please give it a try and feedback! Thanks!

Thanks for your help on this. Using your package, it did get further, but am now running into problems with querying for a specific ProcessDefinition.  Specifically, our code does this:
var processDefinitionList = await _camundaClient.ProcessDefinitions.Query(new ProcessDefinitionQuery
            {
                DeploymentId = deploymentId,
            }).List();

This fails with the info below:
Microsoft.CSharp.RuntimeBinder.RuntimeBinderException : 'Camunda.Api.Client.ProcessDefinition.AutoGeneratedIProcessDefinitionRestService' does not contain a definition for 'GetList'

   at CallSite.Target(Closure , CallSite , Object , ProcessDefinitionQuery , Object , Object )
   at System.Dynamic.UpdateDelegates.UpdateAndExecute4[T0,T1,T2,T3,TRet](CallSite site, T0 arg0, T1 arg1, T2 arg2, T3 arg3)
   at Camunda.Api.Client.QueryResource`2.List() in C:\Users\tom.marlin\git\spok\branches-workflow\aws-workflow\src\Camunda.Api.Client\Camunda.Api.Client\QueryResource.cs:line 20
... (our code here)

Our app uses .NET Core 2.2. I'm not sure if that's causing any of the problems or not.

I get the same problem, I was a bit over-eager  ;(
Fot that I need to delve deeper into Refit (and probably Reflection, urgh) to understand the underlying problem. The change to 5.x obviously brought some significant changes ...

First: a huge thank you to ckadluba for his invaluable help!
After some extensive debugging we found out, that from Refit 4.6.99 onwards QueryResource would not work. The private dynamic _api led to a problem deep inside the framework code (Reflection not being able to handle it correctly and the delegation code than not working).
Since we were unable to make the dynamic property work, we decided to use lambda expressions instead and changed the QueryResource constructor to accept Funcs as parameters instead of an api interface. Updating a "ton" of Service classes we also added a missing GetListCount in one of the History-Services.
This is a non breaking change for the user code, so it should pose no problems with your code. This time it definitely works with our project code (I checked the actual results)!
You can find the latest code at my repository - please give it a shot and report any problems you find!
https://github.com/gaensebluemchenritter/Camunda.Api.Client/tree/master

First: a huge thank you to ckadluba for his invaluable help!

Thank you @gaensebluemchenritter for the nice words. 😊
@Tom-Marlin-Spok could you get the changes from his fork and try them in your application? If the fix works for you too, we could sketch out a PR.

@Tom-Marlin-Spok @Davilink @jlucansky @ckadluba
I tried those changes with the latest version of our own project and it works without a problem. Of course, we are only using a small subset of functions exposed by the API, but getting process definitions, getting running instances, starting new process instances, fetching and locking external tasks, completing external tasks are to me a basis to make the educated guess that this actually works.
Nonetheless additional testing and code review by other people would be great to back this up - so it would be great to get some further feedback before making a pull request

Sadly, we needed a Camunda client for our project and couldn't wait for the this issue to be resolved, so we had to create our own Camunda client.

@Tom-Marlin-Spok @Davilink @jlucansky @ckadluba
Just as a sidenote: did the upgrade to latest Refit version (5.1.67), tested it with our project and pushed it to my master branch (linked above). This gives us the option to upgrade Camunda.Api.Client to Systm.Text.Json instead of Newtonsoft, which rather than not would mean a performance gain and a smaller memory footprint. For new I sticked with Newtonsoft.

@jlucansky @ckadluba @Tom-Marlin-Spok
After initiating a Pull Request (#55) the automated AppVeyor build (using msbuild 15.something) failed with a weird reference to an old version of System.Runtime.CompilerServices.Unsafe that I could not solve for now.
So I had to revert to Refit 5.0.23 for now, which does not yet use System.Text.Json (and therefor abovementioned package).
With msbuild 16.5.something as used by the latest version of VS 2019 there is no problem resolving the dependencies correctly, so perhaps it could be possible to reconfigure the automated build step?


How to locate a launched browser to the specified location?   How to launch several web browsers simultanousely?

Describe the problem
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi, @csxbwang  👋 . I'm Nate, the creator of the Vuplex 3D WebView asset. I'm trying to get a hold of you because you left the following review on my 3D WebView for Hololens asset:

Can you please contact me at<EMAIL_ADDRESS>so that I can help you? As an independent developer, negative reviews can have a very adverse affect on my business. So, your cooperation is greatly appreciated. I just released v3.9 of 3D WebView which may fix your issue, and if it doesn't, I can track it down.

There's not a lot of information in this issue, please reopen with more details

This sounds related to the "Launch browser" buttons in our hand interaction examples scene.
I don't believe specific placement of the resulting app slate is a feature currently supported by the OS.


refactor Stripe webhook

Fixes issue https://github.com/wasp-lang/open-saas/issues/199

@vincanger Pro tip:
When you want to link a pull request to an issue, use GitHub's special words.
This way, once you merge the PR, the issue automatically gets closed.
In this case, you can edit your description to say "Closes #199" or "Fixes #199" instead of "addresses issue #199".
You'll see how GitHub immediately underlines it and add the PR under the issue's title

@sodic @vincanger I wanted to try out myself some things I was suggesting (mostly regarding consolidating types), so that I don't suggest stuff that is not doable or won't look good once it is done, and I got pulled in and ended up implementing stuff.
First I ended up with this PR: https://github.com/wasp-lang/open-saas/pull/217, and then I, on top of that one, made a bit different attempt with this one: https://github.com/wasp-lang/open-saas/pull/218 . I am not sure which one I prefer, maybe the first one, but it is tight: both are a bit clunky in some aspects, but also both work quite well I think.
Please take a look and see if there is anything useful you can use.
Also, since I will be traveling in the following days, and @sodic already got quite involved, I suggest @sodic that you take over the reviewing of this PR and drive it to the end with @vincanger . I think I covered all I could from my side, and you will be able to contribute some extra knowledge on the TS side.

Nice, we got it all in a good shape!

updated docs and app_diff as well


feat: return dynamic data as DartAbi

Changes
Part of #960.

Allows return of arbitrary data using DartAbi as a return type

Not allowed in parameter positions, directly or transitively

Fixes incorrect ffi.Usize type

Checklist

[x] An issue to be fixed by this PR is listed above.
[x] New tests are added to ensure new features are working. End-to-end tests are usually in the ./frb_example/pure_dart example, more specifically, rust/src/api.rs and dart/lib/main.dart.
[x] The code generator is run and the code is formatted (via just precommit).
[x] If this PR adds/changes features, documentations (in the ./book folder) are updated.
[x] CI is passing.

Remark for PR creator

Justfile is a task runner for the command line interface that allows you to run shell commands from a file named justfile in your project directory. You can use Justfile after installing it. Note that commands written in justfile of this repository are expected to be run in bash, not cmd or powershell. Running just ... commands in cmd or powershell will produce errors as the syntax is not compatible. On Windows, you can use git bash if you have Git installed.
If fzyzcjy does not reply for a few days, maybe he just did not see it, so please ping him.

Good job! I will review this later, probably today

Made a review just now, the change LGTM, good job :)


Update Pages “home”

Automatically generated by Netlify CMS

👷 Deploy Preview for parsimony-dev processing.
🔨 Explore the source changes: 9b1e7c189e9628a9dfffb6c17bb3b31c411e9d69
🔍 Inspect the deploy log: https://app.netlify.com/sites/parsimony-dev/deploys/61801a27a88101000726b668


Make Rest Connection abstract and add a CredentialsRestConnection

Make the Rest Connection class abstract in order to remove the need for
a HubServerConfig object to be populated correctly to create the rest
connection.  Create a CredentialsRestConnection that will take the
HubServerConfig object in the constructor and initialize the
RestConnection object. If the username and password are empty then do
not call the setCookies method in the constructor.

@ekerwin @stevebillings @jrichardBD @jessie-anderson Please review the code where you use RestConnection you will need to change it to use CredentialsRestConnection.

Coverage increased (+0.2%) to 64.867% when pulling 84ff101b902276bb91f15f252964c8ade65c8f15 on rest_connection_refactor into aee3febbffe5e91ce0d71c1e0fffe4ee113635c9 on master.

looks good to me.

Great stuff!

Coverage increased (+0.2%) to 64.785% when pulling 81a3f61a26bb1784636ae743161cb5a41550b062 on rest_connection_refactor into 26e1b3a4ee142733b14e89da9a1ee6d446fb49ff on master.

Coverage increased (+0.2%) to 64.785% when pulling 81a3f61a26bb1784636ae743161cb5a41550b062 on rest_connection_refactor into 26e1b3a4ee142733b14e89da9a1ee6d446fb49ff on master.

Coverage increased (+0.2%) to 64.775% when pulling 36d43ccc927ec1a4162885e754f21c82a05de1af on rest_connection_refactor into 26e1b3a4ee142733b14e89da9a1ee6d446fb49ff on master.


Upgrade to Laravel 7

Hello,
In the code we have
https://github.com/cmgmyr/laravel-messenger/blob/master/src/Models/Message.php#L32
without strict types declared, but in the laravel:
protected array $fillable = [];

Hi @zagovorichev,
Sorry, I'm not sure I understand the issue. It appears that Laravel 7's signature is the same. Please feel free to add more information above and a code snippet and I can take another look.

yes, sure, @cmgmyr , sorry, I thought it is clear:
protected $fillable = ['thread_id', 'user_id', 'body']; in your code inherits this:
protected array $fillable = []; from the laravel
so, I've gotten PHP Fatal error:  Type of Cmgmyr\Messenger\Models\Message::$dates must be array (as in class Illuminate\Database\Eloquent\Model) error

Now I'm looking for the place, why do I have this strict types :)

Interesting, it was in the laravel framework v7.7.1, but after the update I have 7.13 and now I need to fix all my code, because there are no strict types declared, really weird :)
thank you for response!

Thanks for the update! I have noticed PHPStorm being a little more aggressive in its reformatting/refactoring actions. Maybe a bug, or intended, but just something to look out for.


PMM-9384 first part

Implemented PMM-T1106, PMM-T486 and PMM-T1104

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-tests/detail/pmm2-ui-tests/11411/pipeline

https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-test-nightly/detail/pmm2-ui-test-nightly/909/pipeline

fixed 2 issues:
https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-tests/detail/pmm2-ui-tests/11419/pipeline
https://pmm.cd.percona.com/blue/organizations/jenkins/pmm2-ui-test-nightly/detail/pmm2-ui-test-nightly/912/pipeline


Update Android test action and use Linux runners instead

This PR performs some maintenance on our Android testing infrastructure. Notably it:

Updates our Android runner action to try and pick up on bugfixes.
Switches the job to run on Linux instead for speed. We enable KVM on the runner as well since the Android emulator needs virtualization and that needs to be fast.
Moves the Android job into its own YAML area so it doesn't get confused with the host OS's tests anymore.

Relates to #68

This PR is ready to go. We are now properly writing device/test logs to stdout and uploading the correct log output path:


feat(localization zh): translate strings

Thanks again @fewwwww! <3
I'm gonna ask the team to review these changes ASAP.
BTW we didn't ask you to use as base branch https://github.com/ardriveapp/ardrive-web/pull/520, so I re-created the PR with the correct base here: https://github.com/ardriveapp/ardrive-web/pull/549
Closing the PR as it's now being tracked at https://github.com/ardriveapp/ardrive-web/pull/549


[ToggleGroup] Deselecting requires two clicks if mapping to/from boolean values

Bug report
Current Behavior
When mapping ToggleGroup values from, and to, boolean values, deselecting the value requires 2 clicks. It can also select the other value if you click them in this order: item 1, item 2,  and then item 2 again.
This is for toggle groups with type "single".
Expected behavior
Clicking the active value once should deselect it.
Reproducible example
Stackblitz reproducible example
Suggested solution
Not sure how to fix it. I tried investigating it and I found that the value returned by useControllableState in line 94 is not updated on the first click.
Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-toggle-group
1.0.4

React
n/a
18.2.0

Browser
Chrome
114.0.5735.198

Assistive tech

Node
n/a
16.17.0

npm/yarn

Operating System

Exactly, I was able to work around it by setting it to null. However, the component doesn't allow for null as the type is string | undefined, it seems more intuitive to use null rather than an empty string.


minor issue - Depends on pycodeexport (but not listed anywhere)

I went through all the steps to install this with pip, but couldn't get it to work
tests worked fine for pycvodes, but failed for pyodesys  (just the native part)
eventually tracked it down to pycodeexport not being installed, and that fixed the issue.

Hi @dlivitz, thank you for reporting this.
It's listed as an "optional depenceny" here:
https://github.com/bjodah/pyodesys/blob/master/setup.py#L81
We could make it a standard dependency because it's just a pure python project. It's needed when using any of the SUNDIALS (pycvodes) or GSL (pygslodeiv2) or Boost (pyodeint) backends, the problem is that I cannot be sure that the user has all of those prerequisites, so I made it an opt in to do e.g.
pip install pyodesys[all]

When I glance over the README I realize that it's not at all apparent, and it should probably be clarified...

Thank you, that makes sense!


Enable test failing due to changes to supported types of main function

Description:
The test MappingConstructorExprTest.testSpreadOpFieldTaintAnalysisNegative is disabled in the master because of the recent compilation checks added to the master to limit the supported types of the main parameters.
Describe your task(s)
Fix or replace the test https://github.com/ballerina-platform/ballerina-lang/blob/3f186a7d32e828ee2d27c2d771c4b3ee4ef18f54/tests/jballerina-unit-test/src/test/java/org/ballerinalang/test/expressions/mappingconstructor/MappingConstructorExprTest.java#L218
Related Issues (optional):
https://github.com/ballerina-platform/ballerina-lang/issues/29829

Enabled on https://github.com/ballerina-platform/ballerina-lang/tree/fix-and-enable-tests in #30281.
Will be merged to master via a separate PR.


I have preview only for "#fff" & "#fffffff", not "rgb(0,0,0)" or "rgba((0,0,0,1))"

I having some issue, it works good for hex color, but rgb & rgba are not displays, why?

Same with hsla()

In pigments by abe33 everything works perfectly, i moved to it.


issue with building the oem partition

Here is the out put that has the error.
from first glance seems that
CMake Error at CMakeLists.txt:68 (find_package):
By not providing "Findresultsync.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"resultsync", but CMake did not find one.
Could not find a package configuration file provided by "resultsync"
(requested version 0.2) with any of the following names:
resultsyncConfig.cmake
resultsync-config.cmake

Add the installation prefix of "resultsync" to CMAKE_PREFIX_PATH or set
"resultsync_DIR" to a directory containing one of the above files.  If
"resultsync" provides a separate development package or SDK, be sure it has
been installed.
-- Configuring incomplete, errors occurred!
See also "/home/mike/libo3d3xx/modules/oem/build/CMakeFiles/CMakeOutput.log".
See also "/home/mike/libo3d3xx/modules/oem/build/CMakeFiles/CMakeError.log".
find_package(resultsync 0.2 REQUIRED) line 68
Where do I get this package.
Or do I even need it.

I cannot seem to find Lib_resultsync anywhere in my filesystem.
Was this something that should have been sourced when I initially build the environment because I am using an older version of  poky-glibc-i686-vantage-image-armv7ahf-vfp-neon-toolchain-qte-1.8.1.sh.
This script was given to me along with the gem tool months ago though.

@mikeNike9 there is no need to rebuild the resultsync library by zour own this is already included in the latest firmware release and within the matching toolchain. So it is maybe needed to update the firmware plus the toolchain. Sorry I can not provide any of them because I am out of office until January the 8th. But zou can contact Ville who will be in the office starting with the 2nd of January.

Closing out stale issues (older than a year). Feel free to reactivate as appropriate.


Calculate G(r) - would like to use Rmax > 100.0

In the Calculate G(r) tab, currently capped at Rmax = 100.0 Angstroms, seen below:

We cannot increase the field to say 101.00. Would like to increase the bound.

Rmax's maximum value is changed to 1000.