Reap all processes if getpid() == 1

This is useful to allow libuv users to run in containers or as init systems (effectively the same thing).

Hi, thanks for the review, that was quick!
Hrm, I think tracking all the children of processes we spawn is very involved. I mean, it might be a really nice solution for use-cases where it's necessary: ie. libuv is a library among many and they are all forking processes, but somehow only libuv is setting up a signal handler (? perhaps you could describe the scenario in more detail because i'm not sure how this is meant to work?).
But, the point is, I think there is also a use-case for a dumber reaper for processes where libuv is the event-loop and process spawner.
The way python asyncio handles all this is by having a child-reaping policy: https://docs.python.org/3/library/asyncio-policy.html#asyncio.AbstractChildWatcher
What's on offer there is:

safe: catch SIGCHLD, reap processes we know about (as libuv currently does)
fast: catch SIGCHLD, reap all processes (the new mode I'm proposing)
pidfd: don't register for signals, listen to events on pidfd's instead (so as not to interfere with other libraries that also want to catch SIGCHLD)
some other, more specialised approaches, but nothing as involved as what you describe although an escape hatch is there for people to implement policy plugins

Would it make more sense for libuv to pursue something more like this? I mean, keep the current default, but opt in to different policies depending on the use-case.

libuv is a library among many and they are all forking processes, but somehow only libuv is setting up a signal handler (? perhaps you could describe the scenario in more detail because i'm not sure how this is meant to work?)

Ah, I think you are talking about the case where some other library runs its own process and expects to be able to waitpid() on it later but if libuv SIGCHLD handler reaps that process then it will lead to erroneous behaviour.
Yes.
Well, it strikes me that this is probably why Linux introduced CLONE_PIDFD, to effectively allow for libraries to create processes which do not generate signals in this way. But that isn't a great answer for libuv which has to support whatever other libraries already exist, as well as windows.

This PR has conflicts and going by the conversation won't get merged as-is anyway so I'll go ahead and close it.


Text tags error

Describe the bug
In the main screen, "Cost Optimisation" and "Performance Efficieny" are shown, and there's one typo in each one
To Reproduce
Steps to reproduce the behavior:
open a generated report.
Expected behavior
A clear and concise description of what you expected to happen.
"Cost Optimization" and "Performance Efficiency" is the correct text
Screenshots
If applicable, add screenshots to help explain your problem.
AWS CloudShell (please complete the following information):

Region: [e.g. us-east-1]
PHP version [e.g. 7.2]
AWS SDK version [e.g. 3.0]
Browser (to display output) [e.g. chrome, safari]

AWS Cloud9 (please complete the following information):

Instance type: [e.g. t2.small]
OS: [e.g. Linux]
EBS volume size [e.g. 20GiB]
Region: [e.g. us-east-1]
PHP version [e.g. 7.2]
AWS SDK version [e.g. 3.0]
Browser (to display output) [e.g. chrome, safari]

Desktop (please complete the following information):

OS: [e.g. Linux]
Version [e.g. 22]
PHP version [e.g. 7.2]
AWS SDK version [e.g. 3.0]
Browser (to display output) [e.g. chrome, safari]

Additional context
Add any other context about the problem here.

Taking a look

Reproduced and resolved.


fix: nav bar issue on smaller screens

fixes #6991
https://github.com/jenkins-infra/jenkins-io-components/assets/107404972/c375d46e-5f90-434d-b030-1384dac1da73

@lemeurherve i don't have vpn installed on this computer, do you know why the build isn't building?

@lemeurherve i don't have vpn installed on this computer, do you know why the build isn't building?

Both infra.ci and ci.j builds are green, I suspect something like https://github.com/jenkins-infra/helpdesk/issues/3888, taking a closer look.

It seems unrelated to the previously quoted help desk, but more to this repo actions settings:

Confirmed in https://github.com/jenkins-infra/jenkins-io-components/pull/123
I'll push the "approve and run" button for this time, let me know if the actions settings should be changed.

@hervelemeur Is this fine?

Uploading nav bar 2.mp4…
working as expected in the local env. Is there some other issue? @lemeurherve

@lemeurherve I ran the local env in firefox where it runs perfectly, while in chrome browser the error arises. Let me fix this

@lemeurherve works perfectly on different browsers, checked with chrome, firefox and edge

Still reproducing with https://deploy-preview-122--jenkins-io-components.netlify.app/?path=/story/example-navbar--searchbox-open on both Firefox and Chrome the strange jump near the limit, from full navbar to navbar on two lines to mobile navbar.

@lemeurherve Sorry for the inconvenience, the deployment works fine for me. Can you force refresh and try again.
https://github.com/jenkins-infra/jenkins-io-components/assets/107404972/3b9ce720-04a1-473e-9e5d-b0b24304a04f

@lemeurherve Sorry for the inconvenience. Do you mean this one?
https://github.com/jenkins-infra/jenkins-io-components/assets/107404972/61b5e249-5258-46cf-bdaa-ab6ee02c1459
former one works fine for me in the deployment site if it doesn't work for you, can you please try again after force refresh
https://github.com/jenkins-infra/jenkins-io-components/assets/107404972/dab7ac32-db8d-4797-8355-eeaf601468cd

@lemeurherve Sorry for the inconvenience.

No worry 🙂

Works fine for me in the deployment site if it doesn't work for you, can you please try again after force refresh

Tried again on FF and Chrome with empty cache and also in private sessions, still the same issue as shown in my previous video.
Does someone else encountered the same behavior?

Do you mean this one?

No.

Does someone else encounter the same behavior?

@zbynek Can you check this and why this happens?

Does someone else encounter the same behavior?

@krisstern can you check this?

still the same issue as shown in my previous video.

The storybook adds a padding around the header component, with that padding removed in dev tools it seems to work fine.

@hervelemeur Is it ready to ship?


Update RishikeshYadav95.json

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Just made changes to the description.

Thank you! Looking forward to getting it merged.


Clear popovers on stream narrows

If you narrow to stream A and open its popover menu, then click on stream B in the sidebar, the popover for stream A still sticks around.   Generally, we solve this by calling popovers.hide_all(); at the start of various UI actions (or at the start of trigger event handlers).   This should be an easy change, with the following minor challenges:

We want to avoid having lots of calls to popovers.hide_all() in our code, so we should be thoughtful about the best place to put these.
When you're manually testing this fix, you should try to uncover similar bugs by doing things like narrowing from the recipient bar, searching, opening up other menus, etc.

Don't know if this is related, but if you press the down chevron and scroll up or down. The popup doesn't exit.

@540KJ I'm not sure I understand; can you post a screenshot or brief video?

As you can see, the popup doesn't belong to (no topic), but rather lorem ipsum. Obtained through opening popup and scrolling up.

Ahh, OK, yeah, that makes sense.  Probably a totally independent issue from the clearing bug; can you open a new issue with that screenshot?

I think this was fixed by #3088.


Merge pull request #3280 from Sage-Bionetworks/release-218

Release 218 into Release 219

This was a bad merge branch.


import instead of require?

Still trying to figure out the whole import vs. require thing - is there a way to use import instead of require like in the examples?

Absolutely. If you use Babel you can do:
import ObjectAssign from 'es6-object-assign';
ObjectAssign.polyfill();

You can also use named imports and do something like:
import { polyfill, assign } from 'es6-object-assign';
polyfill(); // If you want to create the global Object.assign method

// Or you can just use the assign method directly
const obj = assign({}, { foo: 1 }, { bar: 2 });
// { foo: 1, bar: 2 }


Missing Resource

I am trying to use the TStyleBitBtn but when I try to set the Kind property I get the popup: "Resource BBCANCEL not found."
How do I find/use this resource?

I'm working on it: tomorrow I'll push the fix

Fixed in 3.6.6 version.


How can we not introduce external pictures?

I want to upload pictures only from my website, how can I limit it?

Why don't you try addImageBlobHook?


CRAYSAT-1463: Switch default BOS version to v2

Test Description:
Summary and Scope
This change updates the default version of BOS to "v2" in the config
file.
Issues and Related PRs

Resolves CRAYSAT-1463

Testing
Tested on:

Local development environment

Test description:
Run unit tests.
(Functionality relating to BOS v2 has already been tested.)
Risks and Mitigations
N/A.
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable
[x] HPC Product Announcement prepared, if applicable

Thanks for doing this, Jack.
I'm going to go ahead and merge this so I can make my changelog updates without them conflicting with integration as soon as this is merged :)


thuang-fix-logout-redirect-135390

Description
Write a description of your pull request here
Issue
https://app.clubhouse.io/genepi/story/135390/make-the-change-in-the-auth0-redirect-to-route-back-to-homepage-on-logout
Test plan

Visit https://thuang-logout-redirect-frontend.dev.genepi.czi.technology/
Log in
Log out and you should be redirected back to the homepage!

Woohoo! Will test on staging soon 🤞 Thank you!


tekton_pipelines_controller_pipelinerun_count metric counter increases without having any pipeline executed

Expected Behavior
To have the tekton_pipelines_controller_pipelinerun_count metric counter increased only when new pipelines are executed.
Actual Behavior
After a few hours the counter is increased without having any pipelines executed.
Steps to Reproduce the Problem on OpenShift

Run at least one pipeline in the cluster
Expose the metrics endpoint:

$ oc expose -n openshift-pipelines svc/tekton-pipelines-controller --port=9090 --path="/metrics"

Check the counter value:

$ METRICS_ENDPOINT=$(oc get route tekton-pipelines-controller -n openshift-pipelines -o jsonpath="{.spec.host}")/metrics
$ curl -s -k $METRICS_ENDPOINT | grep -v "#" | grep tekton_pipelines_controller_pipelinerun_count

Wait a few hours (4/5) without executing any pipelines, and after re-checking the counter value, you will see it increased

Additional Info

Kubernetes version:

Kubernetes Version: v1.20.0+9689d22

Tekton Pipeline version:

$ tkn version
Client version: 0.13.1
Pipeline version: v0.22.0
Triggers version: v0.12.1

DurationAndCount method:
https://github.com/tektoncd/pipeline/blob/release-v0.22.x/pkg/reconciler/pipelinerun/metrics.go#L126-L164
Where is Invoked:
https://github.com/tektoncd/pipeline/blob/release-v0.22.x/pkg/reconciler/pipelinerun/pipelinerun.go#L199-L206

Yes I confirm this behaviour.
If I install Tekton with a resyncPeriod of the controller = 10 seconds, then when I create a single pipeline, every 10s, the counter tekton_pipelines_controller_pipelinerun_count{status="success"}  is increased by 1.
Looking at the code, it shows also the duration histogram is incorrect. Older pipelines are counted many more times than younger pipelines.
The bug is by design and I think, the only way to remove it is to refactor the code to create this counter only using the lister and not using the controller loop.
If we use the controller loop, we need to use the beforeCondition/afterCondition check to find out if the pipeline was already counted or not. .. But it's not possible: if the controller restarts, we loose this information and the counter will start again at 0.

@wlynch I fixed this in a pr #4468.
@guillaumerose We generally use the rate function of Prometheus query to avoid instance restart related issues.  Even with lister loop, information will be lost during restart because pipelinerun will be deleted by the end-user. So I think before and after condition check is sufficient to resolve this.

/remove-lifecycle stale


[feature]: add pdn model

add pdn model and tests.

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


Small issue in docs

I think there's a minor issue with this snippet in the docs at https://uplink.readthedocs.io/en/stable/user/quickstart.html#response-and-error-handling
def raise_for_status(response):
    """Checks whether or not the response was successful."""
    if 200 <= response.status <= 299:
        raise UnsuccessfulRequest(response.url)

    # Pass through the response.
    return response

I think condition of if describes a success case rather than a failure. Also if response is response of requests library it should probably be response.status_code  (Not sure whether response.status also works or not. I don't think it does.)

Good catch! This should be fixed.

@prkumar I made a PR for this https://github.com/prkumar/uplink/pull/123


OCIS + Authelia + Traefik: Error 401 when uploading a file from the web interface

Using OCIS 5.0.6 docker image, I'm unable to upload any file using the web interface.
I have Traefik in front of OCIS, and Authelia setup as external OIDC. Traefik is configured to forward auth to authelia, and login is working fine, but when I try to upload a file, there is a 401 error in the logs.
I believe the issue is one of the microservice is trying to access to OCIS by using the public url, instead of using localhost, because I see logs in traefik with requests coming from one of my docker ipv6 gateway.
Log from OCIS:
2024-08-06T20:48:04Z INF access-log bytes=0 duration=47.885004 line=github.com/owncloud/ocis/v2/services/proxy/pkg/middleware/accesslog.go:34 method=POST path=/remote.php/dav/spaces/71f13446-3085-4a58-86df-aea050a78676$06c41f70-95b4-4f40-9006-bf215d037e37 proto=HTTP/1.1 remote-addr=<my client IP> request-id=a3fc6fbe-e909-4f43-822c-998b44af3c61 service=proxy status=401 traceid=5465487ca28380b091be737a4da133e6

Log from Traefik:
{"ClientAddr":"[fd9a:78e:85ba::1]:42554","ClientHost":"fd9a:78e:85ba::1","ClientPort":"42554","ClientUsername":"-","DownstreamContentSize":366,"DownstreamStatus":401,"Duration":3143168,"GzipRatio":0,"OriginContentSize":0,"OriginDuration":0,"OriginStatus":0,"Overhead":3143168,"RequestAddr":"<public ocs host name>","RequestContentSize":0,"RequestCount":3380,"RequestHost":"<public OCS host name>","RequestMethod":"PATCH","RequestPath":"/data/eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJhdWQiOiJyZXZhIiwiZXhwIjoxNzIzMDYzNjg0LCJpYXQiOjE3MjI5NzcyODQsInRhcmdldCI6Imh0dHA6Ly9sb2NhbGhvc3Q6OTE1OC9kYXRhL3R1cy9kZDNiODE0NC1hMDg0LTRhMjctYmJkYS1kYjYzYzRlYTE1NDIifQ.Doa-P_xFcB9oV-nBPS_U6zQ_Ki4hgW8bG5pZcpPKewc","RequestPort":"-","RequestProtocol":"HTTP/2.0","RequestScheme":"https","RetryAttempts":0,"RouterName":"to-ocis@file","StartLocal":"2024-08-06T22:48:04.547730109+02:00","StartUTC":"2024-08-06T20:48:04.547730109Z","TLSCipher":"TLS_AES_128_GCM_SHA256","TLSVersion":"1.3","entryPointName":"https","level":"info","msg":"","time":"2024-08-06T22:48:04+02:00"}

This request is forwarded by Traefik to Authelia which denies the access.
Any idea if I can force the service to use localhost for connecting to other service?

@Irda13 Please share some more information about you config. Like e.g. which env vars do you have set for the ocis container.

This request is forwarded by Traefik to Authelia which denies the access.

According to the log line you pasted it has the ocis host name in the request, why should traefik forward it to authelia? Are you sure you traefik setup is fine?


Add the IgnoreErrorsIntegration integration and deprecate the excluded_exceptions option

Q
A

Branch?
2.3

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
yes

License
MIT

Closes #924. I decided that I didn't want to use the $payload argument of the capture* methods as an hacky way to pass options that configured the behavior of the method. Instead, like in the JS SDK I added a new integration that can be further expanded in the future and that will centralize all the logic to decide whether an event should be captured or not according to a set of criteria that must match with the event data. Being an integration means that the configuration now applies to all those methods and not only to the captureException one, so we cannot enable it by default until 3.0

I see that the Go SDK has a similar extension named IgnoreErrorsIntegration and I wonder if I should use the same name as it seems more appropriate and also clearer. Any opinion?

:+1: for that name, I like it

So we'll have to rename sentry.options.excluded_exceptions by sentry.options.ignore_exceptions ?

No, you can leave your code as is and it will keep working until 3.0. However, the excluded_exceptions option has been deprecated in favour of enabling the IgnoreErrorsIntegration that will be applied every time an event is captured, so it will work also when the Monolog handler is involved

so it will work also when the Monolog handler is involved

The excluded_Exceptions wasn't working for the Monolog handler (https://github.com/getsentry/sentry-symfony/issues/273)
As soon as the 2.3 version is released, which of the following sentences is true ?

The excluded_exceptions will work also for the Monolog handler
We'll have to enabling the IgnoreErrorsIntegration if we want to ignore exceptions for the Monolog handler (But how ? With a ignore_exceptions ?)
Some development is necessary for sentry-symfony in order to handle the IgnoreErrorsIntegration.

Thanks !

The excluded_Exceptions wasn't working for the Monolog handler

This is the reason of this PR, to make such option work in all cases regardless of who is calling the Client::capture* methods. To do it, an integration has been created and once enabled it will affect all those methods (instead the excluded_exceptions was not affecting the Client::captureEvent method that was used by the Monolog handler): this is the reason we cannot enable it out-of-the-box until 3.0 or we risk of breaking existing applications. So, at the end of the story:

The excluded_exceptions option is deprecated as of version 2.3 and will be removed in 3.0
The IgnoreErrorsIntegration will take the place of the old option. To use it you will need to use the integrations option to instantiate the class and pass to its constructor the values that you did pass to the excluded_exceptions. You can then stop using the deprecated option and everything should work as before except that this new integration will affect all the Client::capture* methods (and not only the captureError and captureException) and consequently the Monolog handler

Sentry\init([
-    excluded_exceptions => [
-        Exception::class,
-    ],
+    integrations => [
+        new Sentry\Integration\IgnoreErrorsIntegration([
+            Exception::class,
+        ]),
+    ],
]);


memory leak in luigi scheduler task history database code

We are seeing memory issues very similar to #430 , with Luigi 2.6.2. In particular, over time memory usage grows to >6GB, and only goes back down after a restart.
Looking at the code , it seems like self.tasks is storing all the tasks in history, and never gets cleaned. Could we add some kind of process to periodically clean old tasks? Alternatively, could we use some kind of priority queue that removes the oldest entry (past a certain size) on insertion?
@Tarrasch

Hi there, we found a similar problem about memory leaking. After executing about 1000 tasks, the memory usage of luigi will rise to about 50GB. Just like this issue, the only going back is a restart.
the current version we used is 2.6.1, will an update work?
@Tarrasch


Download error

Hi, I am not too sure if I'm doing anything wrong but this is the error I get when executing updater.check()

Any idea?

Looks like this is the error that pops up when there isn't a valid update available on the repo. A "friendlier" error would be better.


Support Option-wrapped lists

Product
Hot Chocolate
Is your feature request related to a problem?
Currently working on #6884 by adapting the type interceptor from this gist. Unfortunately, I seem to be blocked by HotChocolate eagerly prohibiting Option-wrapped lists without actually calling the built-in OptionTypeConverter. That converter converts all option-wrapped values to their underlying representations (or null if the option value is None), and vice versa. However, HC does not invoke the converter and instead returns EXEC_LIST_TYPE_NOT_SUPPORTED whenever I try to query an option-wrapped list (array, in my case, though I believe it applies to all enumerables).
The solution you'd like
For example, make HotChocolate run any converters before determining whether the type is a supported list type.

For output fields, this can be solved with field middleware. However, for input types and arguments, I do not see the ability to use middleware.

field middleware is costly performance wise only use when there is async code on that same field. Never use it to just convert a thing...

It might be that result formatters and input formatters are enough.

Thanks, I'll get a project with unit tests running and let you know.
(By the way, adding the middleware to the field definition seems to make this work for input types and arguments, too. No idea why.)

As mentioned in https://github.com/ChilliCream/graphql-platform/issues/6884#issuecomment-2363787392, a repo is up at cmeeren/HotChocolate.FSharp.
The important code is in Library.fs in the HotChocolate.FSharp project.
The part about unwrapping top-level option-wrapped lists is here.
Considering that that code may be merged into this repo at some point for proper built-in F# support in HC, you are more than welcome to look at the rest of the code in that file, specifically the Helpers module (and the short FSharpNullabilityInterceptor type definition). Additionally, for the same reason, I'd really appreciate some input on how to support Option-wrapped lists inside lists, and Option-wrapped enums inside lists.

Yeah middleware is not meant for this ... try using a formatter .... Definition.FormatterDefinitions formatters work almost the same but are sync

definition.FormatterDefinitions.Add(
            new ResultFormatterDefinition((ctx, result) => unwrap(result)));

each input field btw can have Input formatters ... they exist on both sides ....

Excellent, thank you! I'll have a look at this.
If you have any insight about the other two items (Option-wrapped lists/enums/possibly other cases? and Async<_> support), then I think we'll be in a good place very soon.

I can confirm that worked great, thanks!
Did not need them for inputs. Again I have no idea why it works without it, but I have bigger fish to fry, so I'll leave that.

Just want to state that FSharp.HotChocolate, now available on NuGet, has support for this.
@michaelstaib, I'll leave to you the decision of whether to treat that as sufficient to close this issue, or wait until support is added directly to HotChocolate.

Can we pull the FSharp support entirely and your package deals with this?

cc @glen-84

I guess that depends on what the F# support is? I know about the Option type converter, but not more.
Also, it as mentioned in some other issues, it may require some changes/additions to extension points the library can hook into.

FSharp.HotChocolate now includes OptionTypeFormatter and no longer has a dependency on HotChocolate.Types.FSharp. So if you want, HotChocolate.Types.FSharp can be deprecated with a note to use FSharp.HotChocolate instead
By the way, I think it would be useful to have a doc section or similar about F# where you point users to FSharp.HotChocolate.

OK ... we will remove F# support with V15 and leave it to you.

I however do not think that more extension points are needed ... I think there might only be knowledge transfer needed ... but we can discuss it.

Absolutely, if there are existing ways do accomplish what we need, I'm all ears.

I however do not think that more extension points are needed ... I think there might only be knowledge transfer needed ... but we can discuss it.

@michaelstaib, about that knowledge transfer and discussion:
Since HC 15 removes F# support, I think FSharp.HotChocolate should aim for a feature complete 1.x release by the time HC 15 is out.
I have created a 1.0 milestone that lists the issues where I need this "knowledge transfer" or more extension points. I hope the issue descriptions (or the HC issues/discussions linked to there) are sufficient. Please don't hesitate to let me know if you need more information.

Yes, we have this on our list. However, the next 2 month we will be focused on fusion query planning.


Webpack 4 support

Webpack 4 introduced a new API, including running without any configuration and setting a mode parameter (which is required, I get warnings inside Chrome when running my frontend app).
Are there any plans to upgrade the current default configuration provided by kotlin-frontend-plugin ?

Currently almost everything works well with webpack 4


Crashing with Dancer and Aldrich.

I have a friend who is crashing when trying to enter the Dancers and Aldrich. We worked around Aldrich by putting my sign next to the fog wall, but it crashes on the Dancer when the cutscene ends.
For reference, I think it's because they're on Windows 11 and this mod likely doesn't play well with Windows 11. I don't know if there's anything that can be done, as we've tried everything other than reinstalling the game fresh (because it gets rid of the character if we do that, happened to me several times, happened to them once)

I'm guessing you are using blue sentinel, this is a known bug in it, and there are other closed issues regarding it. This cannot be solved by ds3os, it needs the BS authors to release an update.

Alright, thank you.


Improve GitHub Action

Set timeout-minutes time to 15min
Add on.push.path on.pull_request.path
Raise Python 3.7 -> 3.8
Use cache for actions/setup-python@v3

Ideas taken from https://ashishb.net/tech/common-pitfalls-of-github-actions/

Let's try it.


Schedule shifts needed for each task

Add missing shifts for all task. Depends on Issue #8

Punted to all shifts of 2.5 hours, except last Second Track Liaison shift.


support yaml

I'm using .clang-format file, schemas provided by yamlls, and I have to add schemas manually.
Is it possible to support yaml?
Thans a lot!

Have you seen #3?

🤦♂Sorry didn't search before issue!
It seems yaml.schemaStore.enable just provide the ability for client to ask for a schema list from yamlls. Then client can select one from the list, and set yaml.schemas accordingly. The implementation at client side do need quite a few code, maybe another plugin.
For me, I just want some schemas. I've tried the example code you mentioned in #3, but it didn't work for me. The format of yaml.schemas is different from json.schemas. I did some tweak and it works:
local json_schemas = require('schemastore').json.schemas {}
local yaml_schemas = {}
vim.tbl_map(function(schema)
  yaml_schemas[schema.url] = schema.fileMatch
end, json_schemas)

require('lspconfig').yamlls.setup {
  settings = {
    yaml = {
      schemas = yaml_schemas,
    },
  },
}

It will be convenient if this plugin could provide a builtin support, something like require('schemastore').yaml.schemas.
❤Thanks.


feat(connector): [Payeezy] Currency Unit Conversion

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Modified two files in hyperswitch/crates/router/src/connector/

Addressing Issue #2241
payeezy.rs

Implement get_currency_unit function
Modify ConnectorIntegration implementations for Payeezy

payeezy/transformers.rs

Implement PayeezyRouterData<T> structure and functionality
Modify implementation for PayeezyPaymentsRequest

Motivation and Context

Fixes #2241
How did you test it?

We need to create a payment using payeezy and test the currency unit from connector response and connector dashboard.
Checklist

[x] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[ ] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

I request any of the maintainer to please have a look at the PR and let me know if the changes are relevant.

Hey @adarsh-jha-dev , Pls address the CI fails Run cargo +nightly fmt for formatting.

Hey @swangi-kumari , I am doing these changes on github codespaces due to some unavoidable reasons and it is becoming problematic for me to run the command for formatting. Is there any way you can run that command for me (it's a request) and if not then please give me a little bit of time to do the same.

Hey @adarsh-jha-dev , Pls address the CI fails Run cargo +nightly fmt for formatting.

Done

@SamraatBansal , I request you to please have a look at it and let me know if the changes are relevant?

Hey @adarsh-jha-dev ,
Please address the failing CI checks
Run cargo clippy and addressed lints thrown by it.

Hey @adarsh-jha-dev , Please address the failing CI checks Run cargo clippy and addressed lints thrown by it.

Done, could you please review this .

Hey @adarsh-jha-dev ,
Please address the failing CI checks
Run cargo clippy and addressed lints thrown by it.

Hey @adarsh-jha-dev ,
Your PR still have some compilation errors.
Please address the failing CI checks
Run cargo clippy and addressed lints thrown by it.

Hey @adarsh-jha-dev , Your PR still have some compilation errors. Please address the failing CI checks Run cargo clippy and addressed lints thrown by it.

Yeah yeah , on it, would resolve it within an hour

Hey @swangi-kumari, sorry for not submitting this yesterday, trying my best to submit by tonight. I request you to please don't unassign me

Sure @adarsh-jha-dev , Take your time and address the comments given to you. All the Best:)

Thanks @swangi-kumari for co-operating and giving me time, I have tried hard and resolved those CI fails. I request you to please have a look at them whenever you want.

Hey @adarsh-jha-dev ,
Please make this changes too.
line (95-104)
#[derive(Serialize, Debug)]
pub struct PayeezyPaymentsRequest {
    pub merchant_ref: String,
    pub transaction_type: PayeezyTransactionType,
    pub method: PayeezyPaymentMethodType,
    pub amount: String,
    pub currency_code: String,
    pub credit_card: PayeezyPaymentMethod,
    pub stored_credentials: Option<StoredCredentials>,
    pub reference: String,
}

Here in this struct convert amount to String
pub amount: String,

Hey @adarsh-jha-dev ,
Thanks for your interest in contributing to hyperswitch.
Let us know if you need any assistance from our end.
Also, even if hacktoberfest is over, we should celebrate open source everyday and we are open for more contributions from you.
We would still be rewarding folks with goodies even if the PR gets merged post hacktoberfest.
May the Source be with you!

Hey @adarsh-jha-dev , Thanks for your interest in contributing to hyperswitch. Let us know if you need any assistance from our end. Also, even if hacktoberfest is over, we should celebrate open source everyday and we are open for more contributions from you. We would still be rewarding folks with goodies even if the PR gets merged post hacktoberfest. May the Source be with you!

Yeah, I am almost done with the changes, I am happy to contribute to Hyperswitch even after the fest, it helped me learn a lot of things

Hey @swangi-kumari , I have done the changes requested by you and other maintainers, I request you to please proceed further.

Thanks for your quick response @adarsh-jha-dev,
Looks good to me!

Hey @swangi-kumari , I request you to please have a look at it, whenever you're free

@SamraatBansal  and @Sangamesh26 , could you also please approve this or let me know if there's any change needed?

Hey @swangi-kumari , could you please merge this if you're satisfied with the changes?


[Bug] Missing custom_branch_only from retrieved dbt Cloud job config

Describe the bug
custom_branch_only config is missing from the job definitions retrieved from our dbt Cloud instance.
To Reproduce
Steps to reproduce the behavior:
When running any of the CLI commands from dbt-jobs-as-code other than validate, the following error is encountered:
pydantic.error_wrappers.ValidationError: 1 validation error for JobDefinition
triggers -> custom_branch_only
  field required (type=value_error.missing)

Example of failing command: poetry run dbt-jobs-as-code import-jobs --account-id XXXX or poetry run python src/main.py import-jobs --account-id XXXX
From a debugging session, I put a breakpoint at src/client/__init__.py:139 - the instantiation of the JobDefinition class on that line is throwing validation errors because of the missing triggers -> custom_branch_only fields. Printing the triggers from all jobs within the debugging session shows that the custom_branch_only field isn't present for any of our jobs:
[job.get('triggers') for job in jobs]

which returns the following (truncated):
[{'git_provider_webhook': False, 'github_webhook': False, 'schedule': True},
 {'git_provider_webhook': False, 'github_webhook': False, 'schedule': True},
 {'git_provider_webhook': False, 'github_webhook': False, 'schedule': False},
...
]

Expected behavior
Imported job definitions contain the triggers -> custom_branch_only field and avoid validation errors when building list of JobDefinition classes
Screenshots
NA
Desktop (please complete the following information):

OS: Windows 10
dbt Core Version: 1.6
dbt-jobs-as-code Version: 0.2.0

Additional context
This may be an issue with the dbt Cloud API or an issue with our dbt Cloud account - when running an API request to our jobs endpoint directly (I used Postman), I'm seeing that none of our jobs have the custom_branch_only field present.
We've been using dbt-jobs-as-code within our CI/CD process for about a month now and have the custom_branch_only field present within all of our job definitions, which we retrieved originally using import-jobs and was working until yesterday. We noticed the issue when a PR was opened yesterday and our GitHub action we use to show planned job changes failed - note: we encounter the error above with both v0.1.0 and v0.2.0 of dbt-jobs-as-code

This should be fixed in 0.2.1 now! Let me know if this is not the case.

@b-per - thanks for addressing this so quickly! We're running into the issue now where our planned dbt Cloud job changes are showing diffs due to the custom_branch_only fields for all our jobs:
Jobs are different - Diff: {'values_changed': {"root['triggers']['custom_branch_only']": {'new_value': True, 'old_value': False}}}

If I'm understanding the fix correctly, if no custom_branch_only field is retrieved from the dbt Cloud API for a job, it assumes it's set to True, so the output of plan is showing that it will be set to False for our jobs for which we've explicitly set this field to False.
This isn't a huge deal, but since we're not using custom branches on any of our jobs that we've defined in our job config YML, our planned changes will show this diff until we start receiving actual the custom_branch_only config from the API for our jobs. I think it would be better default behavior if the custom_branch_only were assumed to be False since that's the default setting when setting up a job via the dbt Cloud UI, and I assume it would show fewer changes for others if they're encountering similar issues that we are - I can open a PR reflecting that suggestion while I try to figure out why we're not receiving the custom_branch_only fields for our jobs from the dbt Cloud API, but let me know if you have suggestions in the meantime.

Thanks. I'm ok with it. I will merge the PR and get another release out.


Offset Issues

When trying to edit the offset time for testing purposes I get unexpected results.
the offset returned for the event I specify (sunset) is -18304
when I manually input the offset to bring myself to 18304 the offset returned is -54884. if I set the offset to negative -18304 I get -69723
I think I am not understanding the way the dates presented/configured because when I tick inject all sun times the time for sunset is 2021-03-22T23:10:00.529Z which is not correct for my location, however my automations fire at the correct time...

What time zone you are in?

I think I am not understanding the way the dates presented/configured because when I tick inject all sun times the time for sunset is 2021-03-22T23:10:00.529Z which is not correct for my location, however my automations fire at the correct time...

The times are in UTC. I will think of introducing an option which would allow to generate times in the local time zone.

thanks for the response1
I am in Eastern time zone. is the offset somehow tied into UTC? the logic of it still escapes me

When trying to edit the offset time for testing purposes I get unexpected results.
the offset returned for the event I specify (sunset) is -18304
when I manually input the offset to bring myself to 18304 the offset returned is -54884. if I set the offset to negative -18304 I get -69723

Could you provide more details on how you configured the node? The Offset configuration field is used to shift the message injection time forward or backward relatively to the configured sun event. So, for example, if you set the Offset field to 600 (seconds) and the Sun event field to Sunset, the message is injected 10 minutes (=600 seconds) after the sunset at your configured location. The injected message should have payload.sunEvent property set to the sun event that caused the injection (in the example it would be Sunset) and payload.offset property set to the time offset from the actual event occurrence (in the example it would be 600 seconds). Doesn't it work like described?

I am in Eastern time zone. is the offset somehow tied into UTC? the logic of it still escapes me

The sun event times injected in the message being sent on start-up when you tick Inject all event times after startup option, are sent as Date JavaScript objects. You can print the time of a Date object expressed in your local time zone, for example, by calling toString() method on it (e.g. msg.payload.sunset.toString()).

hey sorry I never responded to this.
The issue im running into is the offset doesnt seem to adjust properly. For example; if msg.payload.offset outputs -3600 with the offset set to 0, when I set the offset for +10 I would expect msg.payload.offset to add 10 seconds indicating that I am now -3610 away from my event triggering.
In reality it appears to set it 20 seconds (or double whatever it is set to) and outputs -3620
See below node and screenshots from my debug tab
[{"id":"b9ce478c.01731","type":"sun-event-inject","z":"79f25591.906644","name":"","latitude":"40.614059","longitude":"-73.728958","event":"sunset","offset":"10","injectEventTimesAfterStartup":false,"x":630,"y":380,"wires":[["7d996383.ee32fc"]]}]

Hi. Sorry for not responding earlier.
I need to ask one more question because I still can't reproduce the issue and I suspect the issue is not a result of a bug in the code but a result of misunderstanding how the node works. Is the incorrect offset observed when you manually trigger the node (i.e. when you click the rectangle on the left side of the node)? If so, then you shouldn't expect the Offset configuration value has any bearing on the value of msg.payload.offset property in the injected message. The msg.payload.offset property express number of seconds to the next occurrence of the configured event at the moment of the message injection. So its value only depends on the time when you clicked the button.


Add search/split iterators for Python

In C++ we have special smart iterators for bulk search and split operations. They lazily report the matches, avoiding heap allocations for the array of match offsets.
For that, an arbitrary matcher (string / character / character set ; in normal / reverse order) is combined with search / split ranges. Similar functionality should be added in Python, where we currently materialize the matches into a "compressed" Strs object.

I'm very interested in contributing to this project as my first step into open-source. I believe I could start by addressing this issue. To clarify, are we aiming to replace the Strs type with something like StrIterator that yields strings lazily? As a first step, should I focus on modifying the split function to eliminate the use of realloc and ensure it returns an iterator instead? Any guidance on this would be greatly appreciated.

Added in 3b6cdddbcba50b326eb44fa799381b978d99bdc5


License unclear

In the absence of an actual license it isn't legally feasible for anyone to use this library. It's some fantastic work and I'd love to evaluate it for a project, but we currently can't.
Is there any chance we could get clarification on the license?

I've licensed it under Apache 2.0.
If you intend on using this in production, this probably needs some code review and a maintainer. There may be better options out there now.


Can we expect an upgrade to 5.7.0?

Running a newer version of Gatsby and really looking forward to using this theme.  Will we be seeing an upgrade to V5.x.x?

I plan to work on it soon, but feel free to start a PR if you want to contribute! That'd be very much appreciated!

With the release of #150 on #153, the theme will support Gatsby v5!
Thanks for the great work, @tekmaven! It wouldn't be possible without your help!

Here's a small migration guide for version 4 of the theme:

Follow Gatsby v4 > v5 upgrade guide (including upgrading to Node 18 and updating packages).
Move the homepage from<EMAIL_ADDRESS>to src/home/index.mdx and add homePath: 'src/home' in the plugin options.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CBOR data cannot be decoded

Description
I have the following CBOR data: 0xA2, 0x01, 0x02, 0x03, 0x26
I try to decode this, using this code:
std::vector<std::uint8_t> v = {0xA2, 0x01, 0x02, 0x03, 0x26};
json j = json::from_cbor(v);
auto j_str = to_string(j);
std::cout << std::setw(2) << j << std::endl;

The code fails with this exception:

parse error at byte 2: syntax error while parsing CBOR string: expected length specification (0x60-0x7B) or indefinite string type (0x7F); last byte: 0x01

However in https://cbor.me/ this data decodes without issues.
This form of maps is used in Fido2, so I consider it valid (check at https://www.w3.org/TR/webauthn-1/#conforming-all-classes, searching for "Example 2).
Reproduction steps
Take the above line so of code and compile it.
Expected vs. actual results
The library is not able to decode the given byte stream - I consider the data as valid CBOR, so it should be possible to decode it.
Minimal code example
No response
Error messages
No response
Compiler and operating system
PlatformIO, GCC
Library version
3.11.03
Validation

[ ] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

I am confused. Shouldn't 0xA2 start a map? Then I would indeed expect a string starting at byte 2.

Hi. Yes, it should start a map! But not a map with "string" to "anything", CBOR also allows a map with "int" to "anything", too.
Here is an example from the Webauthn documentation:

A5
01  02
03  26
20  01
21  58 20   65eda5a12577c2bae829437fe338701a10aaa375e1bb5b5de108de439c08551d
22  58 20   1e52ed75701163f7f9e40ddf9f341b3dc9ba860af7e0ca7ca7e9eecd0084d19c

if you push this trough "cbor.me" you get:

{1: 2, 3: -7, -1: 1, -2: h'65EDA5A12577C2BAE829437FE338701A10AAA375E1BB5B5DE108DE439C08551D', -3: h'1E52ED75701163F7F9E40DDF9F341B3DC9BA860AF7E0CA7CA7E9EECD0084D19C'}

I'm not an expert in CBOR - but it seems that a map does not need to start with a string as key.
So this is seems to be valid "CBOR".

This is not supported, since JSON does not support non-string object keys. See documentation: https://json.nlohmann.me/features/binary_formats/cbor/#deserialization


Add [Phishing] #malicious GDNJAX…JO5DZB

Address: GDNJAXY3KUA6UNKFP3LJAGUWHGO7Y5MYXYRIEABYVV6RUYVPO4JO5DZB
Tags: malicious
Title: Phishing
Requested by @sysmanalex

Stolen XLS, Phishing.

This issue has been closed and anyone with follow up information should refer to the discussion on issue https://github.com/stellar-expert/public-directory/issues/202


Fix dwi reference workflow

I've made the change to replace the reference node based on #1125 (thanks @effigies  for the instruction).  Here is what I did:

copied this function into mriqc/workflows/diffusion/base.py, just above hmc_workflow, named as init_dmriref_wf
removed the dummy scan calculation in init_dmriref_wf, because it's not used anywhere in the current context
changed the docsting to be less BOLD focused.
modified the node dwi_reference_wf
changed the corresponding connections with dwi_reference_wf in the workflow

thank you! I revised it based on your review. One question, what is this Workflow different from pe.Workflow? Because this Workflow was used in the fmriprep; I changed it to pe.Workflow here. Would it make any difference?
https://github.com/nipreps/fmriprep/blob/8d11614685d4522474944a055776d8f88ae56fe8/fmriprep/workflows/bold/reference.py#L25
from nipype.pipeline import engine as pe
from niworkflows.engine.workflows import LiterateWorkflow as Workflow

One question, what is this Workflow different from pe.Workflow?

fMRIPrep uses a subclass of nipype's Workflow which allows us to dynamically construct a description of what processing was applied to the dataset. That's what the __desc__ was about. MRIQC doesn't use that, so using pe.Workflow is correct.

aha, it should be good now. I think there was something in my vscode that automatically created those line breaks....

okay, made some updates based on the reviews

T1w test failure seems unrelated.


Method 'addPostFrameCallback' cannot be called on 'WidgetsBinding?' because it is potentially null.

Error:
/F:/SDK/flutter_windows_2.10.5-stable/flutter/.pub-cache/hosted/pub.flutter-io.cn/another_xlider-1.1.0/lib/another_xlider.dart:441:29: Error: Method 'addPostFrameCallback' cannot be called on 'WidgetsBinding?' because it is potentially null.

'WidgetsBinding' is from 'package:flutter/src/widgets/binding.dart' ('/F:/SDK/flutter_windows_2.10.5-stable/flutter/packages/flutter/lib/src/widgets/binding.dart').
Try calling using ?. instead.
WidgetsBinding.instance.addPostFrameCallback((_) {
^^^^^^^^^^^^^^^^^^^^

I added ? to instance and it works.
WidgetsBinding.instance?.addPostFrameCallback((_) {
      if (!mounted) {
        return;
      }
      _renderBoxInitialization();
      _arrangeHandlersPosition();
      _drawHatchMark();
      setState(() {});
});

https://github.com/loonix/another_xlider/pull/9
Changes were made to clear this issue, should be on the latest version

It will give you error if you are using flutter 2. Upgrade to v3 and check


Rails demo vagrant example in March 30 2020 book as well as github

I'm using vagrant 2.2.6 in a WSL (Windows Services for Linux) Ubuntu environment installed on Windows 10.
Please update the book example of the Vagrantfile on page 241 of the pdf to reflect github:
4 Vagrant.configure(2) do |config|
5   config.vm.box = "geerlingguy/ubuntu1804"
6    config.ssh.insert_key = false
and append a line which is common to your other Vagrantfiles to the Vagrant.configure section:
Vagrant.configure(2) do |config|
config.vm.box = "geerlingguy/ubuntu1804"
config.ssh.insert_key = false
config.vm.synced_folder '.', '/vagrant', disabled: true
Otherwise, the "vagrant up" command fails with a WSL specific error
Thank you

Thanks for the suggestion; I've fixed this in the text and it will be in the next version of the book!


Bump minimum version of Numba to 0.46.0

I keep getting this error. Maybe it's a Windows OS issue based on which job failed?
ERROR: Cannot uninstall 'llvmlite'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.

I keep getting this error. Maybe it's a Windows OS issue based on which job failed?
ERROR: Cannot uninstall 'llvmlite'. It is a distutils installed project and thus we cannot accurately determine which files belong to it which would lead to only a partial uninstall.

that is a pretty old build
likely can’t pin numba there

Got it. Not a big deal if we can't make 0.46.0 the default version. Closing.

confused why can’t u add numba w/o requiring a version?

that build uses numpy 1.15 which is pretty old


chore: delete embedded trace viewer

We decided not to ship the embedded trace viewer. I have some changes to the trace viewer that broke the tests for it, so this PR removes the code altogether.

I was a victim of my own code:
https://github.com/microsoft/playwright/pull/33643#issuecomment-2481249329
It's now time to let it go (well, sort of, because git doesn't let, this is probably the equivalent to purgatory).


Improving ServiceProvider stub and removing trailing spaces

Hey @sarthaksavvy
minor tidy up in the stubs (trailing spaces) and improving the ServiceProvider stub with doc block.
Cheers,
Peter

you are awesome, I will check this and them merge. Thanks

Super small, shouldn't take more than one minute 👍


Keep html tags in question text for CTIS codebook

Description
Keep html tags in question text for displaying in the survey codebook.
Changelog

qsf-tools/generate-codebook.R

@krivard This is ready to merge


hyperline does not show on macOS

I just installed hyper and was eager to add hyperline, but it doesn't show anything or produce any error after adding it to
plugins: ["verminal", "hyperline"],

I've tried reloading and restarting hyper. I'm running on macOS 10.13.4 and Hyper 2.0.0 (stable). Interestingly hyperline doesn't show up in the plugin list in "About Hyper".

Hyperline isn't working on Hyper2.0 yet. See #150

Check #155 for a workaround

Hello, any news on this, thanks.

Seems to be back up and running on Hyper 2.1.0-canary.2 which was released today.
This is a pre-release but seems like a step in the right direction.


two transactionScopes  enclosing one another

Hi,
If a transactionScope with TransactionScopeOption.Required , encloses another transactionScode with TransactionScopeOption.RequiresNew ,
Would this necessarily mean MSDTC will escalate this as a distributed transaction ?
(assuming the connection strings used by the queries used in both scopes are to the same database server )
thanks

Thanks for the question and apologies for the delayed response. Are you still looking for an answer to this question? Did you try a support forum, such as Stack Overflow, or a support chat, such as Slack or Gitter?


feat(CDK Progress bar & spinner): Progress bar & spinner built-in options for displaying percentages and striped designs

Feature Description
similar to other UI frameworks & Libraries, the progress bar & spinner should include built-in options for displaying percentages and striped designs.
Example from Vuetify:

Example from PrimeNg:

Use Case
I'm implementing file upload functionality and need to show a progress bar with percentages. But there is no built-in option therefore making a custom progress bar. So in the future, these should be included.

This isn't supported in the Material Design spec so that's why we don't support it either.


Require CPU Limits

Related Issue(s)

This is Kyverno equivalent policy for below. Setting CPU limits on pods ensures fair distribution of resources, preventing any single pod from monopolizing CPU and impacting the performance of other pods. This practice enhances stability, predictability, and cost control, while also mitigating the noisy neighbor problem and facilitating efficient scaling in Kubernetes clusters. This policy ensures that cpu limits are set on every container.
https://github.com/bridgecrewio/checkov/blob/main/checkov/kubernetes/checks/resource/k8s/CPULimits.py

Checklist

[x] I have read the policy contribution guidelines.
[x] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[x] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

I see I was mistaken. We don't have one which specifically requires CPU limits. Although the claim you make in your post isn't exactly correct, we can add such a policy to the library if you address issues.

DCO is required here.

Please sign off on your PR.

Sign off is not complete.

Converting to draft until all basic requirements are met.


Enable android targets

With the new added targets, openmls will always be ready for Android platform.
Source: https://crates.io/crates/cargo-ndk
https://github.com/bbqsrc/cargo-ndk/blob/main/README.md#L15-L19
rustup target add \
    aarch64-linux-android \
    armv7-linux-androideabi \
    x86_64-linux-android \
    i686-linux-android

Ah, I think it's not actually going to go through because it's from an external branch. I'll just merge as is.


This script appears to convert inherited permissions to explicit permissions

I noticed that the API google provides regarding permissions doesn't seem to distinguish between inherited permissions and explicit permissions.
Thus, when I run this script it converts all the inherited permissions to explicit permissions... this was not what I expected (nor hoped for :-) ).

I'd be curious on how to fix this one.


Custom Resources are not restored

What steps did you take and what happened:
I took a full backup including all the resources on the cluster using Velero.
I created another cluster and installed Velero using:
velero install --provider gcp --bucket <bucket> --secret-file <secret-file> --restore-only
I requested a restore:
velero restore create --from-backup <backup-name> --include-resources --include-cluster-resources --restore-volumes
Velero restores everything (including the corresponding CRDs) except for the Custom Resources. Although the restore status was completed, I waited a couple of hours to see if they would appear. But, they didn't.
Then, I just tried to run the restore command one more time and realized that Custom Resources were restored this time. Of course, it ended with many warnings saying all the other resources have already existed on the cluster.
What did you expect to happen:
Velero restores everything from backup including the Custom Resources.
The output of the following commands will help us better understand what's going on:

kubectl logs deployment/velero -n velero
velero-pod-logs.txt

Environment:

Velero version (use velero version): v1.0.0
Kubernetes version (use kubectl version): v1.12.9-gke.7
Kubernetes installer & version:
Cloud provider or hardware configuration: GCP
OS (e.g. from /etc/os-release):

@suleymanakbas91 it sounds like you're probably running into #964 / #1350 -- TLDR is that Velero doesn't wait for the CRDs to be "ready" after restoring them, so it's possible that by the time it tries to restore the custom resource instances, the CRDs aren't yet ready.
You could do a two-stage restore similar to what you already did -- in the first restore, use the --exclude-resources flag to exclude the custom resources from restore. After that completes and you can verify that the CRDs are ready, do another restore using --include-resources to specify only the custom resources.

Thanks for the suggestion @skriss. As you suggested, I divided the restore process into two commands. First, I restore the CRDs (also services and endpoints for another problem with Istio):
velero restore create --from-backup kyma-backup --include-resources customresourcedefinitions.apiextensions.k8s.io,services,endpoints --include-cluster-resources

After this restore gets completed, I wait for a while to ensure CRDs are ready and then run this to restore the rest:
velero restore create --from-backup kyma-backup --exclude-resources customresourcedefinitions.apiextensions.k8s.io,services,endpoints --include-cluster-resources --restore-volumes

This second restore does not behave consistent. Sometimes it restores all the Custom Resources, sometimes it does not restore some of the Custom Resources. If I run this second command again, it restores all the remainings.
You can find the logs of an example execution here:
first_part.log
second_part.log
second_part_repeat.log
In my first try, it does not restore any of the VirtualService resources.

It sounds like the CRDs may not have been fully ready when the second restore was run, but I'd need to see details of the CRDs to be sure.
I'm going to close this issue as a duplicate of #964 where we're tracking the enhancement request.


Use archunit freezing rules to accept violations

instead of our home-grown solution.

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


findDOMNode was called on an unmounted component

I randomly get this error. I'll investigate and let you know when I find out how to reproduce it.
invariant.js:38 Uncaught Error: findDOMNode was called on an unmounted component.
    at invariant (invariant.js:38)
    at Object.findDOMNode (findDOMNode.js:54)
    at Equalizer.updateChildrenHeights (equalizer.js:73)

Did you manage to find anything? This shouldn't happen since the resize event is removed when the component is unmounted.

I get the exact same error. Happens when I navigate away from a view with Equalizer. Yet, it's only when Equalizer is part of the initial render - not when I navigate to and away from this view (I'm using react-router).

@manumaticx @theisof Try 1.0.6 I have cancelled the debounce when the node in unmounted. Hopefully that should fix the problem you are having.

I have also added a check to 1.1.0 after changing the way the node is referenced.

@manumaticx @theisof are you guys able to reproduce this issue in 1.0.6+ ... is this still an issue..?

Sorry for the late reply (I need to adjust my Github notification settings). I upgraded to 1.2.0 and no longer have the error. Thanks :-)
I think you can close this issue.


Does Swifter persist any data/cache outside of its POD?

Running into a strange problem that I believe is related to Swifter.
I was on a dev branch writing some unit tests (around our networking code, in theory not interacting with Swifter in any way). These unit tests suddenly broke all of our Swifter tests.
The issue persists across branches (master). I also re-cloned the project into a different directory and am observing the same problem.
TLDR: Is there a directory where Swifter stores data outside of the project directory? Or is there a way to reset Swifter?
I realize there's not a lot of information here, I'm still investigating the issue and will update this if I make any additional discoveries.
Thanks.

Running this fixed my issue
https://gist.github.com/maciekish/66b6deaa7bc979d0a16c50784e16d697


change panic(err.msg) to the prefered panic(err)

This is in conjuction to the changes to IError in https://github.com/vlang/v/pull/13041

This can be merged now.

Excellent work! Thanks!


Nuxt with Express Startkit Deploy to Azure Web App

I am trying to get a nuxt with express web app deployed to an Azure Web App running Node.  Here's what I have done so far:

Created a build using VSTS
Deployed build to Azure Web App using VSTS release (using MS's Azure App Service Deploy task)
I also made a few changes per a couple closed issues:

Changed scripts to use fully qualified paths like "node ./node_modules/nuxt/bin/nuxt start" based on response in this issue
Created a stub server.js per response in this issue but I did have to make a change to the call to nuxt.Server to be nuxt.listen(port, host) instead.

Enabled logging to the iisnode log folder based on this iisnode.yml file used in azure
Here is the log information being returned now that makes me think it is still a configuration issue:

Sat Aug 26 2017 16:12:45 GMT+0000 (Coordinated Universal Time): Application has thrown an uncaught exception and is terminated:
SyntaxError: Unexpected token {
    at Module._compile (module.js:434:25)
    at Object..js (module.js:464:10)
    at Module.load (module.js:353:31)
    at Function._load (module.js:311:12)
    at Module.require (module.js:359:17)
    at require (module.js:375:17)
    at Object.<anonymous> (D:\Program Files\iisnode\interceptor.js:459:1)
    at Module._compile (module.js:446:26)
    at Object..js (module.js:464:10)
    at Module.load (module.js:353:31)

Here are the contents of the server.js file I created based on the notes in 3:
const fs = require('fs');
const { Nuxt } = require('nuxt');
const resolve = require('path').resolve;

const rootDir = resolve('.');
const nuxtConfigFile = resolve(rootDir, 'nuxt.config.js');

let options = {};
if (fs.existsSync(nuxtConfigFile)) {
  options = require(nuxtConfigFile);
}
if (typeof options.rootDir !== 'string') {
  options.rootDir = rootDir;
}
options.dev = false; // Force production mode (no webpack middleware called)

const nuxt = new Nuxt(options);

const port = process.env.PORT;
const host = process.env.HOST;

nuxt.listen(port, host);

If need be I can give you the web.config and iisnode.yml files if those would be helpful when looking at the config of the azure web app.

Hi! Just a guess, maybe you have old node.js version and problem line is:
const { Nuxt } = require('nuxt');

@KonstantinVlasov I am using node version 8.1.4.  Would that be the issue?  Sorry I'm new to node and deploying to azure.
I come from a C#.NET background.

@KonstantinVlasov I removed the { and replaced the line with this:
const Nuxt = require('nuxt');

after that change the error is different but I actually made the change to add the { to try and fix this error but I am open to suggestions:
Sat Aug 26 2017 19:01:10 GMT+0000 (Coordinated Universal Time): Application has thrown an uncaught exception and is terminated:
TypeError: Nuxt is not a constructor
    at Object.<anonymous> (D:\home\site\wwwroot\server.js:17:14)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (D:\Program Files\iisnode\interceptor.js:459:1)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Function.Module.runMain (module.js:605:10)
    at startup (bootstrap_node.js:158:16)

Try this:
const Nuxt = require('nuxt').Nuxt;

but looks this is the wrong way, node 8.x is suitable

@KonstantinVlasov ok I think we are getting really close now.
Now I'm back to another error I was seeing but thought it wasn't an issue since it is called UnhandledPromiseRejectionWarning but perhaps this is my issue now.  I found it in the stderr logs not the other error logs.  Now there aren't any other errors being logged that I can see.
Here's the full stderr output:
(node:7308) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): RangeError: "port" argument must be >= 0 and < 65536

Also this might be helpful since this makes me think this is why I'm getting that error.  This is the stdout output:
OPEN  http://localhost:\\.\pipe\650c3970-ae6f-41e5-a62e-1ebec880f2a5

And when I was researching the UnhandledPromiseRejectionWarning it was mentioned that azure uses piped names but I'm not sure how to resolve the issue.

looks like problem is here:
const port = process.env.PORT;

Probably you don't pass port to env.
Here is more safe alternative:
const port = process.env.PORT || 80; // specify your default port

@KonstantinVlasov I agree I think something is wrong with the port #.  I'm not sure why but when I change it to what you suggest it appears to give me the same error.
Running it locally it seems that even though process.env.PORT is undefined it returns true.
Any thoughts around why undefined is no longer falsey?  Did that change in the 8.1.4 version of node?
I'm going to try and hardcode the port to 80 instead.

I think you pass wrong port, just console.log it and check

I console.log'd it and this is what port is returning.  Also found that process.env.HOST wasn't what is used in azure so I changed my server.js to look like this:
const fs = require('fs');
const Nuxt = require('nuxt').Nuxt;
const resolve = require('path').resolve;

const rootDir = resolve('.');
const nuxtConfigFile = resolve(rootDir, 'nuxt.config.js');

let options = {};
if (fs.existsSync(nuxtConfigFile)) {
  options = require(nuxtConfigFile);
}
if (typeof options.rootDir !== 'string') {
  options.rootDir = rootDir;
}
options.dev = false; // Force production mode (no webpack middleware called)

const nuxt = new Nuxt(options);

const port = process.env.PORT || 80;
// eslint-disable-next-line no-console
console.log('port: ', port);
// eslint-disable-next-line no-console
console.log('process.env.PORT: ', process.env.PORT);

const host = process.env.HOST || process.env.WEBSITE_HOSTNAME;
// eslint-disable-next-line no-console
console.log('host: ', host);
// eslint-disable-next-line no-console
console.log('process.env.HOST: ', process.env.HOST);

nuxt.listen(port, host);

And here's the output from the console.log:
port:  \\.\pipe\092d1e59-c81c-4d47-b5b7-357842aac603
process.env.PORT:  \\.\pipe\092d1e59-c81c-4d47-b5b7-357842aac603
host:  kentucky-tourism-frontend.azurewebsites.net
process.env.HOST:  undefined

It appears Azure uses iisnode and autogenerates a pipe for the port every request.  Any idea how to get around this using a named pipe instead of port setup for nuxt?

sorry, I have no experience with Azure or issnode

@KonstantinVlasov ok.
anyone else have experience getting the named pipe way to work with nuxt?

@pmgower Your express integration in the first message is correct (Using const { Nuxt } = require()) Also using node 8.x is great. I think just similar to this the only thing for azure is that we may honor port number assigned by azure: (source)

var port = process.env.port || 1337;
This code is essentially the same as the "Hello World" sample on the nodejs.org website, except it uses the port number assigned by the cloud environment.

So changing your server script to something like this should be the fix:
const port = process.env.port || 1337;

Hi Paul
Have a look at my demo repo, it's deployed in azure with iisnode
It was a pain to figure out, but worth it in the end
https://github.com/shaydoc/nuxtexpress
Check out the
server.prod.js and the web.config
Cheers
Shay

This is it on azure
http://shaynuxtexpress.azurewebsites.net
Cheers
Shay

The only real issue I had to actually resolve in the end was fully qualifying my api calls server side, for the server side rendering.
Cheers
Shay


[SPARK-23340][SQL][BRANCH-2.3] Upgrade Apache ORC to 1.4.3

What changes were proposed in this pull request?
This PR updates Apache ORC dependencies to 1.4.3 released on February 9th. Apache ORC 1.4.2 release removes unnecessary dependencies and 1.4.3 has 5 more patches (https://s.apache.org/Fll8).
Especially, the following ORC-285 is fixed at 1.4.3.
scala> val df = Seq(Array.empty[Float]).toDF()

scala> df.write.format("orc").save("/tmp/floatarray")

scala> spark.read.orc("/tmp/floatarray")
res1: org.apache.spark.sql.DataFrame = [value: array<float>]

scala> spark.read.orc("/tmp/floatarray").show()
18/02/12 22:09:10 ERROR Executor: Exception in task 0.0 in stage 1.0 (TID 1)
java.io.IOException: Error reading file: file:/tmp/floatarray/part-00000-9c0b461b-4df1-4c23-aac1-3e4f349ac7d6-c000.snappy.orc
	at org.apache.orc.impl.RecordReaderImpl.nextBatch(RecordReaderImpl.java:1191)
	at org.apache.orc.mapreduce.OrcMapreduceRecordReader.ensureBatch(OrcMapreduceRecordReader.java:78)
...
Caused by: java.io.EOFException: Read past EOF for compressed stream Stream for column 2 kind DATA position: 0 length: 0 range: 0 offset: 0 limit: 0

How was this patch tested?
Pass the Jenkins test.

Test build #89484 has started for PR 21093 at commit fc5d976.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2417/
Test PASSed.

Merged build finished. Test PASSed.

LGTM

Test build #89484 has finished for PR 21093 at commit fc5d976.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89484/
Test FAILed.

retest this please

Test build #89489 has started for PR 21093 at commit fc5d976.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2422/
Test PASSed.

Test build #89489 has finished for PR 21093 at commit fc5d976.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89489/
Test FAILed.

retest this please

Test build #89507 has started for PR 21093 at commit fc5d976.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2433/
Test PASSed.

Merged build finished. Test PASSed.

Test build #89507 has finished for PR 21093 at commit fc5d976.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89507/
Test FAILed.

retest this please

Test build #89522 has started for PR 21093 at commit fc5d976.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2445/
Test PASSed.

Merged build finished. Test PASSed.

LGTM
Also cc @omalley

Thank you for review, @cloud-fan and @gatorsmile .

Oops. I mistakenly click close and comments button.

Test build #89522 has finished for PR 21093 at commit fc5d976.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89522/
Test FAILed.

retest this please

@dongjoon-hyun Do you have the bandwidth and see why these tests are flaky?

Test build #89532 has finished for PR 21093 at commit fc5d976.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89532/
Test PASSed.

@gatorsmile . Sorry for late response. I'm currently at Dataworks Summit Berlin.
I took a look. It seems that the last two failures are due to JsonInferSchema of BucketedWriteWithoutHiveSupportSuite.

https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89507/testReport/
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89522/testReport/

[info] - write bucketed data *** FAILED *** (4 seconds, 698 milliseconds)
[info]   org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 84.0 failed 1 times, most recent failure: Lost task 0.0 in stage 84.0 (TID 86, localhost, executor driver): java.lang.IllegalStateException: LiveListenerBus is stopped.
[info] 	at org.apache.spark.scheduler.LiveListenerBus.addToQueue(LiveListenerBus.scala:97)
[info] 	at org.apache.spark.scheduler.LiveListenerBus.addToStatusQueue(LiveListenerBus.scala:80)
[info] 	at org.apache.spark.sql.internal.SharedState.<init>(SharedState.scala:93)
[info] 	at org.apache.spark.sql.SparkSession$$anonfun$sharedState$1.apply(SparkSession.scala:117)
[info] 	at org.apache.spark.sql.SparkSession$$anonfun$sharedState$1.apply(SparkSession.scala:117)
[info] 	at scala.Option.getOrElse(Option.scala:121)
[info] 	at org.apache.spark.sql.SparkSession.sharedState$lzycompute(SparkSession.scala:117)
[info] 	at org.apache.spark.sql.SparkSession.sharedState(SparkSession.scala:116)
[info] 	at org.apache.spark.sql.internal.BaseSessionStateBuilder.build(BaseSessionStateBuilder.scala:286)
[info] 	at org.apache.spark.sql.test.TestSparkSession.sessionState$lzycompute(TestSQLContext.scala:42)
[info] 	at org.apache.spark.sql.test.TestSparkSession.sessionState(TestSQLContext.scala:41)
[info] 	at org.apache.spark.sql.SparkSession$$anonfun$1$$anonfun$apply$1.apply(SparkSession.scala:92)
[info] 	at org.apache.spark.sql.SparkSession$$anonfun$1$$anonfun$apply$1.apply(SparkSession.scala:92)
[info] 	at scala.Option.map(Option.scala:146)
[info] 	at org.apache.spark.sql.SparkSession$$anonfun$1.apply(SparkSession.scala:92)
[info] 	at org.apache.spark.sql.SparkSession$$anonfun$1.apply(SparkSession.scala:91)
[info] 	at org.apache.spark.sql.internal.SQLConf$.get(SQLConf.scala:110)
[info] 	at org.apache.spark.sql.types.DataType.sameType(DataType.scala:84)
[info] 	at org.apache.spark.sql.catalyst.analysis.TypeCoercion$$anonfun$1.apply(TypeCoercion.scala:105)
[info] 	at org.apache.spark.sql.catalyst.analysis.TypeCoercion$$anonfun$1.apply(TypeCoercion.scala:86)
[info] 	at org.apache.spark.sql.execution.datasources.json.JsonInferSchema$.compatibleType(JsonInferSchema.scala:271)
[info] 	at org.apache.spark.sql.execution.datasources.json.JsonInferSchema$$anonfun$org$apache$spark$sql$execution$datasources$json$JsonInferSchema$$compatibleRootType$1.apply(JsonInferSchema.scala:262)
[info] 	at

I compared with branch-2.3 itself. Unfortunately, recentbranch-2.3 itself is unstable. There is no success during last 18 runs for SBT builds.

https://amplab.cs.berkeley.edu/jenkins/view/Spark QA Test (Dashboard)/job/spark-branch-2.3-test-sbt-hadoop-2.6/
https://amplab.cs.berkeley.edu/jenkins/view/Spark QA Test (Dashboard)/job/spark-branch-2.3-test-sbt-hadoop-2.7/

BucketedWriteWithoutHiveSupportSuite is testing for Seq("parquet", "json"), and the testsuite fails after insertInto and at write bucketed data. So, the flakiness seems to be irrelevant to this patch. Also, the final one passed 8 hours ago.
[info] BucketedWriteWithoutHiveSupportSuite:
[info] - bucketed by non-existing column (28 milliseconds)
[info] - numBuckets be greater than 0 but less than 100000 (10 milliseconds)
[info] - specify sorting columns without bucketing columns (8 milliseconds)
[info] - sorting by non-orderable column (34 milliseconds)
[info] - write bucketed data using save() (9 milliseconds)
[info] - write bucketed data using insertInto() (9 milliseconds)
... Error starts
[info] - write bucketed data *** FAILED *** (4 seconds, 601 milliseconds)

Thanks! Merged to 2.3

Thank you all! :D


add clock service

Add an IClockService, similar to existing IRandomService.
Call clock service instead of DateTime.UtcNow directly. This abstracts our direct dependency on the machine. It also allows the host to override, which can be very useful for deterministic testing Today(), IsToday(), Now() - especially around UTC , datetime boundaries.

✅ No public API change.


Fixing recaptcha block on animeout with Rest API

Thank you very much for this PR, really appreciated.

Its a pleasure boss, I did more a thorough testing and review of what we have now, downloading multiple anime videos from the host site on different streams and saw that few lines of codes needs to be corrected for a much reliable code base. kindly checkout the new pull request. Thanks.

The person that opened this PR and the person that merged it, God bless you.

The person that opened this PR and the person that merged it, God bless you.

Thanks!


Heap buffer overflow in RenderingServer::mesh_surface_get_lods

Tested versions
current master @ b00e1cbf743dcb6a2b7f6bd14e348a1a7cf3d403
System information
macOS 14.5.0, M2 Pro, Vulkan
Issue description
Related: #98884 , #97862
CC @clayjohn @BlueCube3310
The culprit this time is a 1024x1024 RGTC RG8 normal map, except this time, it seems to decompress normally when testing:
*** Loading texture res://ShakerNormal_decompressed.png
Loading resource: res://ShakerNormal_decompressed.png
*** Decompressing the image
bcdec: Decompressing mipmap 0: 1024x1024, src_ofs: 0, dst_ofs: 0, target_size: 2796202
bcdec: Decompressing mipmap 1: 512x512, src_ofs: 1048576, dst_ofs: 2097152, target_size: 2796202
bcdec: Decompressing mipmap 2: 256x256, src_ofs: 1310720, dst_ofs: 2621440, target_size: 2796202
bcdec: Decompressing mipmap 3: 128x128, src_ofs: 1376256, dst_ofs: 2752512, target_size: 2796202
bcdec: Decompressing mipmap 4: 64x64, src_ofs: 1392640, dst_ofs: 2785280, target_size: 2796202
bcdec: Decompressing mipmap 5: 32x32, src_ofs: 1396736, dst_ofs: 2793472, target_size: 2796202
bcdec: Decompressing mipmap 6: 16x16, src_ofs: 1397760, dst_ofs: 2795520, target_size: 2796202
bcdec: Decompressing mipmap 7: 8x8, src_ofs: 1398016, dst_ofs: 2796032, target_size: 2796202
bcdec: Decompressing mipmap 8: 4x4, src_ofs: 1398080, dst_ofs: 2796160, target_size: 2796202
bcdec: Decompressing mipmap 9: 2x2, src_ofs: 1398096, dst_ofs: 2796192, target_size: 2796202
bcdec: Decompressing mipmap 10: 1x1, src_ofs: 1398112, dst_ofs: 2796200, target_size: 2796202
bcdec: Decompression of a 1024x1024 RGTC RedGreen8 image with 10 mipmaps took 13 ms.
*** Image decompressed
*** Saving the image to res://ShakerNormal_decompressed-test.png

HOWEVER, when the texture is attached to a mesh as a surface material, and we attempt to save the scene as a GLB, the following occurs:
*** Loading the scene res://Shaker_test.tscn
Loading resource: res://Shaker_test.tscn
Loading resource: res://ShakerNormal_decompressed.png
*** Dependencies for res://Shaker_test.tscn:
uid://8eeijsot8t6::::res://ShakerNormal_decompressed.png
*** Instancing the scene
*** Appending the scene to the GLTF document
glTF: Converting light: Light
glTF: Converting camera: Camera
Copying sd.lods[0].index_data, size 98304, lc 49152, rptr[0] @ index_data + 0, past the end? false
Copying sd.lods[1].index_data, size 49152, lc 24576, rptr[1] @ index_data + 2, past the end? false
Copying sd.lods[2].index_data, size 24564, lc 12282, rptr[2] @ index_data + 4, past the end? false
Copying sd.lods[3].index_data, size 12288, lc 6144, rptr[3] @ index_data + 6, past the end? false
Copying sd.lods[4].index_data, size 6144, lc 3072, rptr[4] @ index_data + 8, past the end? false
Copying sd.lods[5].index_data, size 3072, lc 1536, rptr[5] @ index_data + 10, past the end? false
Copying sd.lods[6].index_data, size 1536, lc 768, rptr[6] @ index_data + 12, past the end? false
Copying sd.lods[7].index_data, size 768, lc 384, rptr[7] @ index_data + 14, past the end? false
Copying sd.lods[8].index_data, size 384, lc 192, rptr[8] @ index_data + 16, past the end? false
Copying sd.lods[9].index_data, size 192, lc 96, rptr[9] @ index_data + 18, past the end? false
Copying sd.lods[10].index_data, size 96, lc 48, rptr[10] @ index_data + 20, past the end? false
Copying sd.lods[11].index_data, size 48, lc 24, rptr[11] @ index_data + 22, past the end? false
Copying sd.lods[12].index_data, size 12, lc 6, rptr[12] @ index_data + 24, past the end? true

=================================================================
==39550==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x000159ea7d48 at pc 0x000115fc4960 bp 0x00016b0fb210 sp 0x00016b0fb208
READ of size 2 at 0x000159ea7d48 thread T0
    #0 0x115fc495c in RenderingServer::mesh_surface_get_lods(RID, int) const rendering_server.cpp:1724

The problem is here:

The uint8_t index_data.ptr() is cast to a uint16_t ptr rptr, and then it's read at rptr[i].
Thus, rptr[12] is at index_data + 24, and since sd.lods[12].index_data is only 12 bytes long, it reads past the bounds and causes the crash.
Full asan log:
thing.log
Steps to reproduce

Optionally, put the following in servers/rendering_server.cpp @ line 1722:

print_verbose(vformat("Copying sd.lods[%d].index_data, size %d, lc %d, rptr[%d] @ index_data + %d, past the end? %s", i, sd.lods[i].index_data.size(), lc, i, i * 2, i * 2 >= sd.lods[i].index_data.size() ? "true" : "false"));

Build the editor with sanitizers enabled
Extract the MRP somewhere.
Run the MRP with <editor_bin> --path <wherever_you_extracted_the_mrp_to> --verbose
Observe crash.

Minimal reproduction project (MRP)
bcdec-crash-mrp.zip

mesh-lod-crash-mrp.zip
Was able to trigger a crash (with sanitizers enabled) in a gameplay scenario by attaching a mesh with "bad" index_data to a soft body. It has nothing to do with bcdec or textures, this is a completely stand-alone mesh.


System.IO.FileNotFoundException: 'System.Drawing.Primitives Error - occures after .netCore Web app has run for some time correctly

I have .netcore MVC application with one MVC and two blazor projects. The app uses Select.HtmlToPdf.NetCore (22.2.0) to generate PDF files.
This works fine (sometimes for a week, sometimes only for one day) and than, just of a sudden, I get the following error:
System.IO.FileNotFoundException: Could not load file or assembly 'System.Drawing.Primitives, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified. File name: 'System.Drawing.Primitives, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'
The app is on version 7.0.5 and deployed (framework independent) on a windows server using IIS.
This was working fine up to verison 7.0.0 and started sometime after I updated the security updates.
Any idea what could cause this´?
projectFile.txt

I have the same issues with the following assemblies:
System.IO.Pipes
System.Security.AccessControl
System.IO.Compression;
System.Drawing;
Running locally works fine (dotnet run), however, when I publish my app (ASP.NET Core 7.0.5) I get the same exception (System.IO.FileNotFoundException: Could not load file or assembly) for the assemblies mentioned above.
As workaround, I do the following on Program.cs:
var pipesAssembly = typeof(System.IO.Pipes.PipeStream).Assembly;
var accessControlAssembly = typeof(System.Security.AccessControl.AccessRule).Assembly;
var compressionAssembly = typeof(System.IO.Compression.ZipArchive).Assembly;
var drawingAssembly = typeof(System.Drawing.Bitmap).Assembly;
Please note my project references different projects (around 10), most of them also net7.0.  Only 2 are netstandard2.0, not sure if that might be root of the issue, however, I cannot change other projects using netstandard2.0.
When I deploy the mode I use is Self-contained


fix(audio): fix audioTextResponse decode

The audio api passes audioTextResponse to sendRequest,
	if request.HasJSONResponse() {
		err = c.sendRequest(req, &response)
	} else {
		var textResponse audioTextResponse
		err = c.sendRequest(req, &textResponse)
		response = textResponse.ToAudioResponse()
	}

but it cannot be asserted as *string type, lead to decoding failure.
func decodeResponse(body io.Reader, v any) error {
	if v == nil {
		return nil
	}

	if result, ok := v.(*string); ok {
		return decodeString(body, result)
	}
	return json.NewDecoder(body).Decode(v)
}

Fix this problem.

@WqyJh thank you for the PR! Could you also please add a test for that?

Added test.

@WqyJh thanks! There are some linter problems now


Stack

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes:  #540

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist:

[x] I have made this from my own
[ ] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Done !!


Works great, but disconnects momentarily every so often.

Heya! I love your software so far! Just wanted to give some feedback and see if it it helps improve its compatibility.
Issue one is that it disconnects momentarily every so often, (about every half an hour, maybe less) and it refuses to register keypresses for a few seconds until returning to normal.
I'm using a wireless MyArcade Super gamepad, so it may be something to do with how the wireless dongle works.
Secondary issue is that unplugging and re-plugging just ends up locking up the controls at times. The controls in gamepad settings go all wonky until I unplug the dongle.
I have to reflash the adapter with your software to get the controller working again.
I suspect it's an unexpected USB disconnection that is the culprit.
Other than that it's super responsive and works really well on an arduino pro micro.

Thanks for the nice words. To be honest, I haven't used this code at all after writing it, so I don't think I could help you with your issues.
I don't know what would cause the "disconnects", and I've only ever tested it with a few of the official, wired controllers.
I understand the issue regarding unplugging, since I didn't implement any detection of disconnecting either side of the adapter. The Wii extension port does apparently have a device detect pin, but I didn't know how to implement that. You shouldn't have to reflash the adapter; just turning it off and on should be sufficient.


Symptom Checker- Desktop and Mobile-Missing Landing Page and Disclaimer Page from Figma MVP

Landing page of Symptom checker needs to be updated to match Figma Design MVP (https://www.figma.com/file/4cV84H1jyMo0rJEAl3HNnJ/FightPandemics-landing-page?node-id=4147%3A17391).
Also missing the Disclaimer page from MVP

Just to make sure: The page is supposed to match this now?

Hey @nadinejuraschek , in Figma, there are two pages prior to this page that I think need to be implemented in staging. Currently, we are sent directly to this page.

#400 will fix the mobile of this, need to add desktop after this is merged.

Working as expected. Closing this.


How import CryptoPouch with Typescript ?

import PouchDB from 'pouchdb';
import PouchdbFind from 'pouchdb-find';
import CryptoPouch from 'crypto-pouch';
PouchDB.plugin(PouchdbFind);
PouchDB.plugin(CryptoPouch);
let db: any = new PouchDB('datalayer');
db.crypto('password').then((res: any) => {
console.log(res);
});
this doesn't work for me :/
in the browser I have the following error
Uncaught TypeError: Cannot read property 'then' of undefined

Howdy! That code should work with crypto-pouch 4.0.0. Previously, db.crypto() was not asynchronous, which is why it didn't have a .then() method -- it wasn't a promise!
Let me know if you're still having issues :)

@Jusmaar there's a type definition now.
$ npm install --save-dev @types/crypto-pouch


[Investigation] Expose simplified metric models to reporters than than valuesource metrics

Leaving this as is, better to expose values sources in a generic way and it also provides value providers to do things like reset on get


Implement a much more efficient scrollback buffer - which only prints the required output to reconstruct a session

Blockers to merge:

Needs lots of tests
Need to merge vt100 changes to upstream.
Need to make the number of scrollback lines configurable in ~/.config/tab.yml
Need to make sure this works on: Ubuntu, NixOS, Void Linux, OSX iTerm, OSX Alacritty, OSX Terminal
Need to make sure this works with: vim, nano, kak, top, ytop, btm

Thanks for the workaround suggestions! What solutions are other terminal multiplexers using for this problem?

Tmux seems to use some cell datatype (if I’m correct) and it keeps track of which ones to draw at all times, per pane.

Left: alacritty with neovim
Right: alacritty with neovim in tab
For some reason coloring doesn't work?


Search Jobs: Location filter is not working correctly

The location filter in the search jobs tab is not working correctly and needs to be fixed.
I also think it will be helpful if the filters selected are showcased instead of looking at blank text boxes in the input fields.

Search results are now correctly producing filtered results. Location filter is now working as intended.

Changed Adzuna URL to now reflect US based search results by default instead of Great Britain.