reload region cache when store is resolved from invalid status (#843)

Cherry-pick #843 to tidb-6.5.

cannot build.

Hold this PR because it may spawns too many goroutines.

See the discussions in https://github.com/tikv/client-go/pull/843.
TODOS for the release-v6.5 branch:

Make the change about acccessFollower.next simple to reduce risk, verify the correctness.
Verify there's no extra or hidden cache reloading logic in the RegionCache, the background async reload worker is the only entry to resolve the region actively.

What's wrong with golangci-lint 🤔
run golangci-lint
  Running [/home/runner/golangci-lint-1.47.3-linux-amd64/golangci-lint run --out-format=github-actions] in [] ...
  panic: load embedded ruleguard rules: rules/rules.go:13: can't load fmt

  goroutine 1 [running]:
  github.com/go-critic/go-critic/checkers.init.22()
  	github.com/go-critic/go-critic@v0.6.3/checkers/embedded_rules.go:47 +0x4b4

  Error: golangci-lint exit with code 2
  Ran golangci-lint in 12405ms

@crazycs520 @ekexium @zyguan
PTAL


imgpkg copy should be re-runnable for registries that disallow mutating tags

Describe the problem/challenge you have
Based on the following slack conversation: https://kubernetes.slack.com/archives/CH8KCCKA5/p1626878622031300
There appears to be errors when re-running imgpkg copy with the same bundle, targeting a registry that does not allow mutating tags.
Error received:
TAG_INVALID: The image tag 'sha256-ABCDEF.image-locations.imgpkg' already exists and cannot be overwritten because the repository is immutable

Describe the solution you'd like
Ensure re-running imgpkg copy with the same input works on registries that dis-allow mutating tags.
Anything else you would like to add:
linked thread was using version: v0.14.0

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

There appears to be errors when re-running imgpkg copy with the same bundle, targeting a registry that does not allow mutating tags.

One (big) caveat: it doesn't appear to be a consistent issue. Much of the time, rerunning imgpkg copy works fine.
It's possible there is some kind of non-determinism going into the image-locations generation, but there doesn't seem to be a good way to grab the image-locations.yml that imgpkg is attempting to upload to confirm this.

Hi @gabegorelick
I have spent some time trying to get oci-location to 'mutate' across runs. But I have been unable to do so.
Check out this branch for a e2e test https://github.com/vmware-tanzu/carvel-imgpkg/tree/try-find-oci-location-mutate-exploration
Are you able to provide more information:

bundle / nested-bundle structure
-- images.yml for each bundle / nested-bundle

From the slack convo:
We were provided with the following info https://kubernetes.slack.com/archives/CH8KCCKA5/p1627569740185300?thread_ts=1626878622.031300&cid=CH8KCCKA5:
---
apiVersion: imgpkg.carvel.dev/v1alpha1
images:
  - image: ghcr.io/external-secrets/kubernetes-external-secrets@sha256:76b1e4948ddd1bac562248bac9fe2149b9b96d1b319fb7cf1c6ad07e9d13e5e1
    isBundle: false
  - image: index.docker.io/fluent/fluent-bit@sha256:8e3fdc011d3ff98502143650dfa3261860b11f1f289dbb3184c630f71236a366
    isBundle: false
  - image: k8s.gcr.io/autoscaling/cluster-autoscaler@sha256:1c19fa17b29db548d0304e9444adf84e8a6f38ee4c0a12d2ecaf262cb10c0e50
    isBundle: false
  - image: k8s.gcr.io/descheduler/descheduler@sha256:61b14f675870d0c2c1b692b99c16662245a23334434662a2820665c8b2557268
    isBundle: false
  - image: k8s.gcr.io/metrics-server/metrics-server@sha256:6c5603956c0aed6b4087a8716afce8eb22f664b13162346ee852b4fab305ca15
    isBundle: false
  - image: k8s.gcr.io/node-problem-detector/node-problem-detector@sha256:072be07890487b187ddf181f9418b4ff92979b70e5acf5a98e39b7dfbfc0ee08
    isBundle: false
  - image: public.ecr.aws/aws-ec2/aws-node-termination-handler@sha256:20e99a648895d2378c1c0656c0464e38f96023012a1aecf6014ffef3be5fa4b7
    isBundle: false
  - image: quay.io/bitnami/sealed-secrets-controller@sha256:cb183c914323dcad01a46e6c879368b2f4706a2c021a9386fb6a6a5e8768fa37
    isBundle: false
kind: ImageLocations

We've confirmed that the order of images in ImageLocations is changing between invocations of imgpkg copy.

fix is included in https://github.com/vmware-tanzu/carvel-imgpkg/releases/tag/v0.17.0


Fix huawei bug

This PR is for:

Server

Fixes huawei bug
Steps to reproduce the bug

learning nat gateway with no network info

Changes to fix the bug
Affected branches

main

Checklist

[ ] Added unit test to verify the fix.

 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.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Use VRS so that I can find voice experiences from companies that I’m interested in

Problem #2: As a consumer, I want to use VRS so that I can find voice experiences from companies that I’m interested in and discover new voice experiences across the different voice platforms.
Submitted by @rogerkibbe

Meeting 09.03.2020:

Consumer means user.
Voice experiences mean voice application

Problem #2: As a user or technical resource(Publisher/Developer), I want to use VRS so that I can find voice applications from publishers across different conversational platforms.
Example: Find all the voice apps from Target across all different conversational platforms.
Separate use-case:

Technical resource(Publisher/Developer) - can be a representative of an entity or independent party.  Their role to create an actual listing of the voice application.


[10.0] syncStripeStatus trying to update incorrect status column

syncStripeStatus method from Subscription model trying to update status column which is not correct. Correct column name is stripe_status.

Good catch. Thanks!


[AIRFLOW-658] Improve schema_update_options in GCP

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-658

Testing Done:

Ran changes against the existing BigQuery hook and operator tests and it passed.  Also did local tests that uploaded data from GCS to BigQuery using the schema_update_options field and also without using the field.  Everything worked so it should be good to go.

Current coverage is 66.76% (diff: 100%)

Merging #1909 into master will increase coverage by 0.17%

@@             master      #1909   diff @@
==========================================
  Files           136        136
  Lines         10130      10144    +14
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6746       6773    +27
+ Misses         3384       3371    -13
  Partials          0          0

Powered by Codecov. Last update 6fb9463...6ab0a6b


filestore support broke

Today Barry Marson and I were trying to install Ceph filestore with ceph-ansible and it wasn't creating OSD filesystems correctly.   We were using Ceph Luminous from download.ceph.com.  We tracked this down to the fact that we had to add the "--filestore" option to ceph-disk command used for the osd_objectstore: filestore case in this line of code.  Did the default for ceph-disk change to --bluestore recently?  If so, note this probably has to change wherever ceph-disk is run (i.e. multiple scenarios) to create a filestore OSD.

Yeah, ceph-disk in luminous defaults to bluestore.  In ceph-deploy we
added a --filestore option (to complement --bluestore) to force a
particular choice, but by default it's bluestore.  I think ceph-ansible
should be the same (i.e., default to bluestore unless a specific backend
is specified).

Yes, that's fine with me if ceph-disk defaults to bluestore, but my only concern was that ceph-ansible should still allow you to select filestore if you request it.

@bengland2 a PR #1676 has been submitted for that issue

thank you for the awesomely quick response.

follow up here: https://github.com/ceph/ceph-ansible/pull/1704


Installing setup.py

I hope you realize how helpful and amazing this creation is here! Thank you! I am however having a hard-time installing setup.py. I was wondering if you could simply clarify the setup process? If time is allowed of course. If I click on the python file, a console flashes open and closes. I have tried running it from within a python console and even went as far as specifying the file directory for setup.py. I am going to continue fighting this battle but, I figured if it wouldn't hurt to through this out there for any others having issues.  :)

Hello - This library is designed to be run as part of a web server/service. It's not going to be very useful if you're trying to run it through a file browsing GUI.
While there is no requirement that your entire app runs on Python, you will need some experience working with Python to use this library. A full walkthrough of this process is beyond what I'm able to support here. Best of luck with your project.

ok, Thank you.


Add assertion amin, ...?

The following results in a segmentation fault:
#define XTENSOR_ENABLE_ASSERT 1

#include <xtensor/xarray.hpp>
#include <xtensor/xrandom.hpp>
#include <xtensor/xmath.hpp>
#include <xtensor/xio.hpp>

int main()
{
  xt::xarray<double> A;

  std::cout << xt::amin(A)[0] << std::endl;
}

It would help debugging if there would be an assertion firing.

Thanks. However, the updated example still results in a segmentation fault.

Alright, then we are missing some checks in the code!

I quickly looked but could not so quickly decide where and how to check.

Actually that's weird, the check is correctly done in xreducer::element
This needs further investigation, either we need some additional checks before (and that would be specific to the xreducer, or there is a bug in the check itself). I'll have a look today.

Yes what I meant is the segfault you have without defining XTENSOR_ENABLE_ASSERT is due to a bug in the stepper, the one you have when you define XTENSOR_ENABLE_ASSERT is due to a bug in the check_element_index. One issue for 2 bugs, pretty good ratio ;)

I see.
Maybe this gives me the right to remove one item of my wall of shame ;)

Well I think you can remove two of them, I found a third bug when fixing check_element_index! See #1294

Even with the current master, I still get a segfault on
#define XTENSOR_ENABLE_ASSERT 1

#include <xtensor/xtensor.hpp>
#include <xtensor/xarray.hpp>
#include <xtensor/xrandom.hpp>
#include <xtensor/xmath.hpp>
#include <xtensor/xio.hpp>
#include <xtensor/xmanipulation.hpp>

int main()
{
  xt::xtensor<int,1> a = {};
  std::cout << xt::amin(a)[0] << std::endl;
}

So I think that this issue should be reopened.

The issue was automatically closed when the PR was merged, reopening it; the issue is specific to empty xtensor, the code does not segfault when a is an empty xarray.

@JohanMabille That I can reproduce.

I actually wonder if that should work for uninitialized tensor of dimension higher than 0. When not initialized, an xarray is considered as a 0D array holding the default value_type so calling a reducer on it makes sense, it is similar as calling the reducer on a scalar.
However xtensor<int, 1>() is an empty 1D tensor, not holding any value. I'm not sure that it would make sense to call a reducer on an empty container.

I would say that
sum([]) == 0
sum([[]], axis=0) == []


remove RSS url's until RSS functionality works

:+1:  Thank you!

@anthonny Thank you for the merge!


Fix re-render children

Fix re-render children when change page,
The problem is arrow function creates new every render, so the children will re-render
ref: https://reactjs.org/docs/faq-functions.html#arrow-function-in-render

@nwalters512 I had clean this PR, pls recheck it. Thank you 😊

Ping @nwalters512

Im still having trouble with children causing rerenders although I've changed the page component to extend React.PureComponent.
Is there anything else I'm missing?

@ndom91 Can you provide an simple example?

I tried to create a codepen, but its all pretty deep in an internal project so I'll try to describe it.
In the top level layout.js wrapper class component I have a header component which should stay static, and the page changes should transition below it.
I have the {this.props.children} underneath the header component now wrapped in the <PageTransition> component.
The children in this example case, is a class component (inbox.js) which I've changed to PureComponent. The problem is, when for example when a modal opens due to a state changes inside this inbox.js page class pure component and it rerenders it also forces the <PageTransition> wrapper in the layout.js component above it, to refire.
I hope that made sense. Let me know if theres anything I can elaborate on.

Super old issue but I got around this in my use case by storing the pid in state, which may/may not be useful to others:
const router = useRouter();
const { pid } = router.query;
const [product, setProduct] = useState<Product>();

useEffect(() => {
  if (!pid) return;
  setProduct(products[pid as string]);
}, [pid]);

if (!product) return null;


Make sure NFT is not susceptible to the common vulnerabilities that were found in fungible token example

There are some common vulnerabilities and inconsistencies that were found in our fungible token example, many of them are not specific to whether the token is fungible or not fungible. We need to make sure NFT does not have them.
Issues to review:

https://app.zenhub.com/workspaces/chainmiddleware-5cea2bcf78297c385cf0ec81/issues/near/core-contracts/47
https://app.zenhub.com/workspaces/chainmiddleware-5cea2bcf78297c385cf0ec81/issues/near/core-contracts/54
https://app.zenhub.com/workspaces/chainmiddleware-5cea2bcf78297c385cf0ec81/issues/near/core-contracts/55
https://app.zenhub.com/workspaces/chainmiddleware-5cea2bcf78297c385cf0ec81/issues/near/core-contracts/56
https://app.zenhub.com/workspaces/chainmiddleware-5cea2bcf78297c385cf0ec81/issues/near/core-contracts/57

CC @evgenykuzyakov

without further information, we have no other option but to close this issue


feature: add disable_mouse_event, closes #5913

Signed-off-by: Shogo Hida<EMAIL_ADDRESS>Fixes #5913

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

@lucasfernog CI failed with Windows builds which seems to be irrelevant to this PR. Could I merge this one?


Modify interfaces of services and modules that touches the native crash workflow

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#761  👈
#760
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite


Add endpoint for predefined catalogue item properties #233

Description

Add collection for catalogue item property templates
Add get endpoint for getting all catalogue item property templates

Testing instructions
Add a set up instructions describing how the reviewer should test the code

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage
[ ] {more steps here}

Agile board tracking
closes #233

Quick question: How are these templates going to be added?

Quick question: How are new templates going to be added?

It is going to be imported like the units. However I have time to implement a full CRUD (frontend and backend) for this before MVP. I can start the implement of this after I have done the UI changes for the migration of properties which I'm starting today.

It is going to be imported like the units. However I have time to implement a full CRUD (frontend and backend) for this before MVP. I can start the implement of this after I have done the UI changes for the migration of properties which I'm starting today.

If this can be delivered as it is then I would rather address the full CRUD after the MVP. I think we discussed that this and other related functionality make sense to be in a separate service since it is only of use to the frontend but not the IMS API itself.

#245


Split English into UK and US variants

The "English" text has a mix of American and British terminology and spellings
Tire/Tyre
Program/Programme
Deck lid/Load compartment
Hood/Bonnet
Gas lock/Fuel flap lock
Distance traveled/Distance travelled
I have replace the en.json file with two variants en_uk.json and en_us.json

Hi, I guess I misunderstood your point with the suggestion to split the en into 2 variants. In the last commit I added the en-GB.json file. Please change the strings for the GB variant and create a new PR. The other files are auto-generated, so leave them as they were. I will add this translation for GB to card, thanks a lot.

No @ngocjohn, you didn't misunderstand, I spotted a few more differences as I started editing so thought I would make the two variants. It doesn't help that MB use US spellings for their API! I'll do what you suggest now. Just happy to help.


Version 0.32.0 seg faults on exit

Hyprland Version
Hyprland, built from branch HEAD at commit 751d2851cc270c3322ffe2eb83c156e4298a0c0e dirty (props: bump ver to 0.32.0).
Bug or Regression?
Bug
Description
When I press META+M (close Hyprland) it crashes
How to reproduce
Compiled latest version from git it started to crash on exit.
Crash reports, logs, images, videos
hyprlandCrashReport111057.txt
hyprland.conf

is this still happening for you ?


Facebook Callback appends '#=' to Return URL

When authenticating with facebook and having this route:
router.get('/auth/facebook/callback',
        passport.authenticate('facebook', {
            successRedirect : '/',
            failureRedirect : '/'
        }));

.. callback somehow appends #_=_ to redirect URL.
How can this be fixed?

I know in angular you can set html5 but other than that im not sure.

This is coming from facebook's end. More info here:
https://github.com/jaredhanson/passport-facebook/issues/12#issuecomment-61456906

Related issue

Closing.  Not an issue with this project.

I know this answer is late, but the following code works (at least for me):
TL;DR:
if(location.hash=='#_=_')location.replace(location.href.split('#')[0]);

More detailed version:
//If #_=_ was appended to the URL:
if(location.hash == '#_=_'){
  //Store the "clean" URL in a variable
  var cleanURL = location.href.split('#')[0];
  //Set the URL to the "clean" URL
  location.replace(cleanURL);
};


CircleCI error with the credentials

CircleCI builds do not run because of invalid credentials. When I build in CircleCI on my user account they work, but when we merge with master the builds are failing because of invalid credentials. Can we fix this?

@MarshallOfSound is the only one with access to CI, so he would need to fix it. It appears its broken for darwin builds as well.

Not only for Darwin build. If you go to CI you will se that Darwin build is not running. The first failing build is that for Linux. All next builds are not running.

I think it wouldn't hurt, seeing as GitHub has Actions now, to get the CI converted to workflows. I'm going to leave this open as the issue for that, and label it as such so maybe someone with more free time can take it on.


About licensing

It seems you forgot to add licensing stuff, such as:

A file LICENSE at root.
License header for each source file.

:-)

Good call -- just applied the MIT license to make the code free as in speech and, of course, beer.


Replace cromwell part I

Using wdlTools to replace the Cromwell parser, type-checker, and evaluator in the dxWDL project.

@jdidion it would be nice if you took a look. Much of this is to make the definitions closer to they they are in Cromwell. It makes it easier to replace Cromwell in dxWDL.

LGTM


[feature] change  kind 'Cluster' to 'MysqlCluster'

Is your feature request related to a problem? Please describe.

currently our kind is cluster, this is confict with kubesphere cluster.

Describe the solution you'd like

change  kind Cluster to MysqlCluster
Describe alternatives you've considered

Additional context

same as #236


Screen sharing during call does not work

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: https://docs.mattermost.com/install/troubleshooting.html.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary
Sharing screen or window doesn't work during a call. This works with Firefox 101.0.1 and Chromium 95.0.4638.54 on same client and server thus I assume it's an issue of the Desktop App.
Environment

Operating System: Ubuntu 20.04 LTS
Mattermost Desktop App version: 5.1.0
Mattermost Server version: 7.0.0

Steps to reproduce

Start call -> Works
Click "Start Presenting" Icon -> Dialog pops up
Select what to share (screen/window) and click "share" --> Nothing happens

Expected behavior
Screen/Window is shared and a preview of the shared content being shown above the small call dialog in the bottom left of the App (as in Firefox/Chromium).
Observed behavior
Nothing is shown as preview, screen/window is not shared.
Possible fixes
N/A

@torse Would you be open to moving this issue to the Calls repo https://github.com/mattermost/mattermost-plugin-calls/issues? When you do so, please also help include the following details if possible:
1. JS Console Logs

Dots menu -> View -> Developer Tools for Current Tab.
Right Click -> Save as…

2. Call Stats Dump
Running the /call stats slash command in the channel the call is ongoing. A JSON object will be returned to the user through an ephemeral message.

Closing as this is a Calls issue


Regression: Hot reloading not working in development

Expected behavior
When saving a file in snapshot-hub the vue express server should reload. I think this may be a regression from https://github.com/openlawteam/snapshot-hub/commit/5b7bcf7d41beb2ae359e38b81975f7fc042a97ac.
Actual behavior
When saving a file it now does not reload. You must close the docker session and restart manually.
Steps to reproduce the behavior
Start the local development server then try saving a file.

my bad @jtrein ! Just reverted that commit. Thanks for flagging.

Thank you!


How to Ignore hidden elements

I have dependent fields in the form. How to ignore hidden fields from validation.

Hidden fields are ignored by default. Are you seeing a different behavior?

Yes, Depentdent fields not ignored. Check this fiddle. https://jsfiddle.net/Dhruvang/pvnheqy2/4/
When you select Yes in first question, second question will be hide. But it is not ignored.

@dhruvangg, only input type=hidden fields are ignored. It does not look for hidden elements (I'm not sure if it should at this moment). But there's one way to solve the issue in your case: remove the required attribute from the dependant attribute and re-initialize the Pristine object. In any case, you have to re-initialize the Pristine object if any attribute is modified.

Hi, can you add some function for re-initialize for example reset()

@1ucay, you can basically do the following to re-initialize:
pristine.reset(); // resets errors
pristine.destroy(); // clean up
pristine = new Pristine(form); // re-init

If you have dependent/dynamic fields in a form, than try using combo of hidden attribute with disabled. Than, you can pass something like this
const pristine = new Pristine(form)
const activeInputs = (form) => [...form.elements].filter(input => !input.disabled && input.type !== "hidden"))
pristine.validate(activeInputs(form))


Fine grained polling

The current implementation re-runs the entire processing on all files which will slow down when working with huge sets of files.

[Moderate] Create threads for the processing to keep the speed in-tact
[Easy] Add listeners on each file to poll every second to compare the last modified time of the file and process just that file (might have to refactor for handling stylesheets from user with the fine grained )

Both of the above are requirements that need to be done but aren't dependent on each other so can be 2 different tasks.
Branch out accordingly

Done with unsafe pointers,
Still have to change to using refs to improve this


Fix demo app

Fixes #51
The synchronization is an easy-fix solution and we still need to discuss if there's a better solution to this.

I personally think this issue would happen for other people too, so I added an issue on Javalin: https://github.com/tipsy/javalin/issues/423


Gitlab Discovery Processor should automatically add the project-id if not present in catalog-info.yaml / backstage.yaml

Feature Suggestion

The @loblaw-sre/backstage-plugin-gitlab depends on the gitlab project-id to be added to the entity in a specific location. It would be great if the discovery processor could automatically populate that id into that location so that users don't have to specify it in their yaml files. If the user does specify a project-id in the yaml file, the plugin could skip the attempt to populate it so that it doesn't override the value set by the user.
Possible Implementation

I don't have context into the discovery processor to provide implementation suggestions, but I know there might be alternatives to the gitlab discovery processor to populate those entities, and it would be great if this same feature could be implemented by them as well.
I believe these changes would live in this plugin:
@backstage/plugin-catalog-backend-module-gitlab
Context

https://github.com/loblaw-sre/backstage-plugin-gitlab/issues/154
Step 5 in the Setup Guide outlines the location in the yaml

If I understand the situation correctly, the discovery processor does have knowledge of what the project ID is, but later down the line when the UrlReader wants to read the contents of the generated Location entity, this knowledge is lost because it's not encoded in the URLs. Is that a correct assessment?
If so, then yeah, something needs to be added that lets this contextual information be passed down through the layers. Not sure what the best solution would be, off the top of my head, but suggestions are welcome.

For context, I see this comment in the discovery processor: https://github.com/backstage/backstage/blob/45681c215b456a88f7b3a915898a6ee4a02f4cbf/plugins/catalog-backend-module-gitlab/src/GitLabDiscoveryProcessor.ts#L153

The format expected by the GitLabUrlReader:
https://gitlab.com/groupA/teams/teamA/subgroupA/repoA/-/blob/branch/filepath
This unfortunately will trigger another API call in getGitLabFileFetchUrl to get the project ID.
The alternative is using the buildRawUrl function, which does not support subgroups, so providing a raw
URL here won't work either.

This is how the code then fetches the project ID based on the URL where the file is stored:
https://github.com/backstage/backstage/blob/45681c215b456a88f7b3a915898a6ee4a02f4cbf/packages/integration/src/gitlab/core.ts#L103
This is something that anybody can do at a later time, for example a processor that fills in these IDs in annotations in a pre-processing step. Unfortunately that would lead to more API hammering of your GitLab instance - but the calls could be heavily cached, so maybe that actually can be sidestepped to a large degree.
To be clear, the UrlReader can't have this type of knowledge, since it's way more low level and should not modify data.

A solution that currently works very well with the backstage-plugin-gitlab plugin is using 'gitlab.com/project-slug' instead of 'gitlab.com/project-id'.
After adding the AnnotateScmSlugEntityProcessor to your catalog configuration like this
  builder.addProcessor(
    AnnotateScmSlugEntityProcessor.fromConfig(env.config),
  );

the project-slug annotation is automatically added for all entities originating from a GitLab repository.


fix fix NPE of hive meta store client

Proposed changes
The failed to connect to hive meta store, the exception will be thrown.
But there is a bug that the exception object may not be set, causing NPE.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Recommended approach to API Authentication?

Hi!
I've been working on understanding the pinning concept, which I understand to be highly important for keeping content available on the network "in a permanent manner".
We have a thread going here: https://discuss.ipfs.io/t/trying-to-better-understand-the-pinning-concept/754
Ultimately what I've realized is that content added via a local daemon can be pinned on a remote VPS IPFS node (therefore making the content "permanent while the node is up") via the official API described in this repo, but unfortunately there's no API auth built in here (yet?)
Potentially NGINX can handle the reverse proxy, and Authentication, allowing one to pin content remotely, but I'd like to know the recommended approach here?

There is https://github.com/ipfs/go-ipfs/issues/1532 which is something we want to do in future but currently there isn't much of a recommended practice.

Cool - thanks @Kubuxu !
Do you see any reason why using NGINX for API Client Auth would be an issue?
http://nginx.org/en/docs/http/ngx_http_auth_request_module.html#auth_request
https://www.nginx.com/blog/authenticating-api-clients-jwt-nginx-plus/
https://github.com/auth0/nginx-jwt


BottomDrawerFragment has padding for horizontal

Description: BottomDrawerFragment has padding for horizontal
Expected behavior: It should be empty by default.
To help us triage faster, please check to make sure you are using the latest version of the library.
We also happily accept pull requests.

Yes, indeed. I'll get down to it soon. 😁

Cool! +1 for the nice work.

Thanks, @dolphpire-manager


V1.5: Upgrading

Allow upgrading a orevein so it can produce faster and produce more.

Warning: The only way easiest that I can see to make it upgrade-able is to modify it's meta size and speed/max_time values. (However, this does mean it will reset to it's original values once picked up, thus loosing all upgrades)

[x] Make Upgrade items.
[x] Trigger on_use when pointing at a oreveins node.
[x] Apply "upgrade" to node and consume item from player.
[ ] Track upgrades spend and allow refunding the player.
[ ] Modify on destroy for oreveins to return itself and it's upgrades.

Either way I want to make a GUI for oreveins to enable mine-ability (can you/someone else mine it), just to add a safe guard for those who want it on say a public mine.

[ ] Allow toggling mine-ability.
[ ] Possibly allow toggling operation (So you could turn it off if you don't plan on needing anymore right now)

Limit upgrades

[x] Set limit for size (I'm thinking max size of 3 or 4, since that becomes a 7x7x7 or a 9x9x9 which is rather large)
[x] Set limit on speed (I'm thinking the fastest any orevein should be allowed to go is from anywhere around 5 to 10 seconds, leaning towards maybe 8)

Patch:

[ ] Fix it so oreveins return upgrades only if upgrades were performed on the vein. (Currently as of v1.4 there are upgrades but the veins always return at least 1 of both upgrades, even though upgrades can't be upgraded)

Patch:

[ ] Fix it so oreveins return upgrades only if upgrades were performed on the vein. (Currently as of v1.4 there are upgrades but the veins always return at least 1 of both upgrades, even though upgrades can't be upgraded)


Allow Kibana's Elasticsearch public URL to be configurable

This patch adds a new configuration variable in order to allow the
public elasticsearch URL that Kibana uses for its javascript bits to
be configurable directly without affecting any other services.
The variable, elasticsearch_public_url, may be set in any of the
/etc/openstack_deploy/user_*.yml files.
Closes-Issue: https://github.com/rcbops/rpc-openstack/issues/586

+1

:+1:


1.2.1

Coverage remained the same at 99.671% when pulling 41b52eef0e22a821535f0ec885485ad1d4ccaaf5 on 1.2.1 into 97dde45174537c48366c6e945e502764e1d00578 on master.


Add Throne of Misc favour while on island.

I just recently made the switch from osb, they have an option to show misc favour while on Miscellania. Would be cool if Runelite could get that as well. Thanks for the great client.

We already have a plugin for this, its just disabled by default


fix: use DataTransferStatus in status

In ocpp 1.6, the field status is of type
DataTransferStatus, as we have it defined,
this is what we should be using for the
status field of DataTransferPayload.

Fixes #223


Updating feature dependencies inside environment breaks PATH

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
pixi.toml:
[project]
name = "pixi-test"
version = "0.1.0"
description = "Add a short description here"
authors = ["twrightsman <snip>"]
channels = ["conda-forge"]
platforms = ["linux-64"]

[dependencies]

[feature.feat1.dependencies]
marimo = "~=0.4.11"

[environments]
env1 = { features = ["feat1"] }

Reproduction
$ pixi shell -e env1
(pixi-test:env1) $ marimo --version
0.4.11

Now, bump marimo to marimo = "~=0.5.0" in pixi.toml.
(pixi-test:env1) $ pixi install
✔ Project in <snip>/pixi-test is ready to use!
(pixi-test:env1) $ marimo --version
bash: marimo: command not found
(pixi-test:env1) $ echo $PATH
<snip>/pixi-test/.pixi/envs/default/bin:<snip>
(pixi-test:env1) $ exit
$ pixi shell -e env1
(pixi-test:env1) $ marimo --version
0.5.1
(pixi-test:env1) $ echo $PATH
<snip>/pixi-test/.pixi/envs/env1/bin:<snip>

Issue description
Pixi appears to set the PATH incorrectly during pixi install when a feature's dependencies are updated. This does not happen if the dependency is in the default feature. Using pixi install -e env1 shows the same behavior.
Expected behavior
The above "command not found" should not occur.

Found the issue fixed in #1364

That's awesome! Thank you for debugging this issue.


Change the alternative for matrix from nmatrix to numo-narray

Hello.
I came to this site from the badge in the README of matrix.
I noticed that nmatrix is proposed as an alternative to matrix.
Unfortunately, nmatrix has not been maintained for several years. Today's Ruby machine learning library experts, such as Andrew Kane and Atsushi Tatsuma, build their tools using numo-narray rather than nmatrix.

rumale
torch.rb
https://ankane.org/numo

In my benchmarks numo-narray is faster than nmatrix.
Please consider changing.

Awesome, thanks for letting me know


Build failure: pkgsi686Linux.ncurses5

Steps To Reproduce
Steps to reproduce the behavior:

nix build nixpkgs#pkgsi686Linux.ncurses5

Build log
error: builder for '/nix/store/qrblidcrxqb8mb1llys1rah4r33ipxmc-ncurses-abi5-compat-6.4.drv' failed to produce output path for output 'dev' at '/nix/store/qrblidcrxqb8mb1llys1rah4r33ipxmc-ncurses-abi5-compat-6.4.drv.chroot/nix/store/qjy34z05zcnl9bvv9p99mz9ws06ykk4l-ncurses-abi5-compat-6.4-dev'

Additional context
I stumbled upon this Issue in https://github.com/NixOS/nixpkgs/pull/307983#issuecomment-2086270230, much more context can be seen in that PR
Notify maintainers
! split ncurses into dev output: @vcunat
last commit in ncurses: @ConnorBaker
also output related commits in ncurses: @dezgeg
added i686pkgsLinux.ncurses to android build tools: @Artturin @hadilq
Metadata
- system: `"x86_64-linux"`
 - host os: `Linux 6.6.25, NixOS, 24.05 (Uakari), 24.05.20240408.4cba8b5`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"home-manager-23.11.tar.gz, nixos-23.11"`
 - nixpkgs: `/nix/store/450afzqlzzgw6wnyc3dwysf3i5yxyqkr-source`

Add a :+1: reaction to issues you find important.

I never touched ncurses5 package, but if I had time I would run git bisect to find the commit that caused this problem. If you don't have time, I'll do it in the next two weeks.

I might be able to do it next week, not sure though..
Weird thing is, as discussed in #307983, that seemingly building androidenv platformtools sometimes results in that build crash, for some not..
I'm not that familiar with nix yet, but thats a rather weird occurance to me, might it be that pkgsi686Linux.ncurses5 was buildable once and @FlafyDev still has it cached in his system?
I currently circumvent this problem by removing pkgsi686Linux.ncurses5 from plotformtools build inputs.
does nix build nixpkgs#pkgsi686Linux.ncurses5 at least result in the same error for someone else or might it be a problem with my system?

nix build ".#pkgsi686Linux.ncurses5^*" builds for me and dev contains the files it should @ d88e2d5 (current master)

Ok, thanks for letting me know :)

Do nix build nixpkgs#pkgsi686Linux.ncurses5 --rebuild

error: some outputs of '/nix/store/qrblidcrxqb8mb1llys1rah4r33ipxmc-ncurses-abi5-compat-6.4.drv' are not valid, so checking is not possible

You didn't have a succeeding build, so --rebuild doesn't work.  You just run normal build again to retry.

You haven't had a succeeding build yet, so --rebuild doesn't work. You just run normal build again to retry.

in the issue they say nix build nixpkgs#pkgsi686Linux.ncurses5 fails with the same error

I don't know.  I'm unable to reproduce the issue.

Hopefully it works by now.


Cache then network example doesn't use service worker

The recipe currently shows how to do this on the main thread, which is interesting, but I think a more real world use case of this would be doing this in the service worker.
Thoughts?
@TimAbraldes

The recipe currently shows how to do this on the main thread, which is interesting, but I think a more real world use case of this would be doing this in the service worker.

My thinking was something like: "Page gets info from cache (not requiring SW), then initiates fetch. Page needs to be notified when fetch replaces entry in cache; page might as well have fetched directly from network instead of through SW"
For a single page, there's not much reason to move logic into a SW. In a real site there would be multiple pages, so we would want to centralize the "fetch into cache" logic. That leads to a solution something like:

Page uses fetch to get resource
ServiceWorker responds with cached data and initiates network fetch
When network fetch completes, if the resource is updated, update the cache and post a message to clients indicating what's been updated


Go

Provide a simple install script to make it easier to install go with diverse package naming across distros and lack of packages at all (e.g. on Ubuntu).

This seems out of scope for ebash.  Thoughts?

This seems out of scope for ebash. Thoughts?

That's fair. It's just super convenient :-). There are lots of other things that ebash provides the ability to install that have nothing to do with ebash (e.g. install/docker) that are opt-in scripts should people want to use them. Ebash itself doesn't install go or require you to install it. It's just trying to make it easier to do so.


Reorganize modules

This PR reorganizes submodules (as suggested in https://github.com/crossbeam-rs/crossbeam/issues/179#issuecomment-405053632).
r? @stjepang

@stjepang I reorganized things as you suggested.  Thanks!
I think it's better to merge this PR after releasing a new version of crossbeam-utils, as after the crossbeam-utils 0.5.0 release this crate should be updated accordingly.

.. and now this PR bumps the version of crossbeam-utils it depends to 0.5.0!

This is more of a hypothetical question, but here it goes: While we currently don't have any basic synchronization primitives, we'll probably implement them in the future. I wonder what module they should belong to.
Some examples of such primitives:

Exchanger
StampedLock
Semaphore
Simple latches
Handoff<T>, which is basically just an Option<T> and AtomicBool, which can be used for passing a single value from one thread to another. Very useful (example).
Scalable reader-writer lock, where readers don't contend by updating the same atomic value on every lock (bad for cache efficiency).

I'd probably put those in sync module (we don't need it right now), but I wonder what you think.

cc @Amanieu
It seems some of the examples @stjepang mentioned are already in parking_lot.  Personally, I think those primitives are fit into parking_lot rather than crossbeam.  Though maybe we can re-export parking_lot as a submodule of crossbeam.

parking_lot is a mechanism to support blocking threads while waiting on a particular memory address. I can definitely see a place for Semaphore in there. Not sure about the others, they seem more obscure.
Regarding StampedLock, have you looked at my seqlock crate? It seems to be similar.

On second thought, you might be right -- such utilities could be out of scope of crossbeam. And, we already have crates like seqlock and semaphore.
Anyways... As far as I'm concerned, after adding utils module this PR will be ready.

Thanks for the review.  I'm merging it!


Change AWSReader methods to return maps rather than slices

For convenience of not having conflicts with so close open PRs, this branch is on top of the change of #25
All the context of this PR is in #16 and the commit messages should contain enough information of the changes applied.
It's worth to mention that I haven't changed the returned types by the AWS as I mentioned #16 as a further improvement, mostly because some of them could not be changed because they contain more than one field and others only one field (which could be changed), hence for homogeneity I decided not to change only the ones that we could.
closes #16

Coverage increased (+4.1%) to 84.473% when pulling bd92e01055d44ec207b7422b69204e11cc5ef6e5 on if-16 into b7afb8027a69b9717c00909807d143424b9dac21 on master.


⚠ Processors has degraded performance

In bc763dd, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 59 ms

Resolved: Processors performance has improved in 530acd5.


Add proper CFI and other inline assembly cleanups

The most significant change is that all of the inline asm for x86 and x86_64 now has proper CFI annotations, which allows gdb to produce a correct backtrace even if the program is interrupted in the middle of a swap operations.
I didn't do this for the OR1K code because I am not familiar with the instruction set and don't have any way of testing the resulting code.
I've also added a few cleanups for the inline asm clobbers.

@Amanieu Also, looks like you broke OS X. This is likely related to the DWARF encoding, I suggest looking with objdump --dwarf=frames to see which encodings are generated.

I've addressed all the comments. I tested both x86 and x86_64 on Linux, but I haven't been able to test on OS X apart from the tests run by travis.

LGTM pending inline comments and or1k port.

I've fixed the comments. I think the or1k port is best left to a separate PR for now.

@Amanieu is there a problem with it?

Not really, it's just that it's hard to make sure I don't accidentally break it if I have no way of testing it.

Just tell me once you have something and I'll test (and fix if something is broken).

I reworked the trampoline code so that it is compatible with frame pointer-based unwinding. I personally feel that this approach is superior to #44 since it preserves the guarantee that a backtrace can be correctly generated at all instruction boundaries.

This is now covered by #49.

I'm reopening this since it seems that the approach in #44 is broken. You actually do need 2 trampolines.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 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.:x: VitoDegirolamo sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Ship our embedded YAMLs and check they're up-to-date

We would like dependent projects to be able to use (indirectly) our
CRDs in particular. Doing so requires shipping them as code, so we
need to publish yamls.go.
The linting jobs are updated to check that yamls.go is up-to-date.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

The GHA is verified by https://github.com/submariner-io/submariner-operator/runs/5008922791?check_suite_focus=true (which correctly failed).


Version 4.17.42 After closed, view mode will go back to days mode

In line 936, Version 4.17.42, there is currentViewMode = 0;
As a result, after closed, view mode will go back to days mode

Thanks, I've been searching the source of this bug !

4.17.43 still have the same code. Is it just remove this code and then it will be fixed?

Yes. I'm using 4.17.42. Just comment out currentViewMode = 0;

Yes, it works too after I comment it.

Yes. I'm using 4.17.43. commenting out currentViewMode = 0; did not work for me

.43, still have the same issue.

+1 for .43

Hi, this was solved on latest version.
https://jsfiddle.net/William_/ja6v6x2x/


Simplify angular bootstrap

Codecov Report

Merging #1099 into master will decrease coverage by 0.34%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1099      +/-   ##
==========================================
- Coverage   94.07%   93.73%   -0.35%
==========================================
  Files          39       39
  Lines        4269     4339      +70
  Branches      468      740     +272
==========================================
+ Hits         4016     4067      +51
+ Misses        253      252       -1
- Partials        0       20      +20

Impacted Files
Coverage Δ

browser/model/devstudio-autoinstall.js
97.29% <0%> (-2.71%)
:arrow_down:

browser/model/helpers/hash.js
85.29% <0%> (-2.59%)
:arrow_down:

browser/pages/install/controller.js
79.85% <0%> (-1.73%)
:arrow_down:

browser/model/helpers/downloader.js
96.13% <0%> (-1.66%)
:arrow_down:

browser/pages/account/controller.js
82.11% <0%> (-1.33%)
:arrow_down:

browser/model/jbossfusekaraf.js
95.09% <0%> (-0.83%)
:arrow_down:

browser/services/data.js
91.5% <0%> (-0.79%)
:arrow_down:

browser/pages/confirm/controller.js
87.59% <0%> (-0.78%)
:arrow_down:

browser/model/cdk.js
99.24% <0%> (-0.76%)
:arrow_down:

browser/model/installable-item.js
93.47% <0%> (-0.71%)
:arrow_down:

... and 17 more

Continue to review full report at Codecov.

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


[DOCU-536] Add Run in Insomnia button for Konnect

Review
@falondarville
Summary
Adding "Run in Insomnia" button, generated with https://insomnia.rest/create-run-button.
Reason
To make it easier to use the Konnect API, and to cross-advertise Insomnia.
Testing
https://deploy-preview-3116--kongdocs.netlify.app/konnect/reference/konnect-api/

For future reference: tested Insomnia v2021.3.0, v2021.4.0, and v2021.3.1. Turns out this was a new feature with v2021.4.0.


Review shared logic between models and consolidate where possible

Now that we've added 5+ models, I think worth revisiting shared patterns and consolidating some of that logic.
I think this issue can also account for refactoring our internal streaming logic to be more unified across models 👍
One other thing I noticed - we could consistently prefix model specific structures with their name. Ex, we do a good job of this in mistral.py where we have things like from mistralai import TextChunk as MistralTextChunk, but we don't do this in openai.py.
We can unify:

Docstrings
Perhaps some sort of protocol for private patterns like _map_messages that could ultimately turn into an accessible hook

(WIP)

@YanSte,
Sure, help always welcome!
I think this will sort of be a WIP as we finish up internal refactors (specifically, streaming is next).


Encoding Problem

Hi
First of all I wanna thank you for this great extension I was trying to implement sth like this then I hit this BApp of yours.
I'm having trouble mocking a http response which contains ISO-8859-1 characters and when I just copy and paste my desired body and check the response I see weird stuff (encoding wired chars) exactly where those characters exists. I'd be grateful if you could consider adding support of multiple encodings in your application because technically not all websites are English. Strangely enough I couldn't even reverse the encoded message to its original format (It might be doubly encoded or sth)
I might know how to solve my own specific problem though I think adding a .encode(iso-8859-1) before StringToByte method (in converting string to byte for assembling ResponseMessage to send) might do it however, I don't know how to compile your source code to get the jar file.
If you need I can provide ambiguous encoded message compared to its original format for further analysis.

I'm glad the extension has been helpful to you.
Do you have an example of an Burp extension (https://github.com/PortSwigger) that supports this properly in your experience?
Can you give an example of text that would exhibit the issue and reproduction steps that will show the issue in Reshaper when this text is used? Should be able to attach a file containing the text here.
FYI, I have added development instructions to the Readme.

Hi thanks for your reply
Honestly, I haven't tried many BApps and I don't know any other specifically which supports multiple encoding.
Sure, I'm uploading 2 text files which contains corrupted body and original one and a picture of RULES tab.
I wonder why I couldn't find building instruction LOL
I'm trying to apply the solution I mentioned in previous post, and I will keep you informed about the outcome.
Basically as an example the word "حداکثر توان" which is Persian gets converted to "-/'©+1 *H'F"
TrueBody:
https://0bin.net/paste/aFYL7umh#x43B5kYTX1NeQ9bSscm9de+TMNaNb-0MWtHOJ6FqXFA
Problematic:
https://0bin.net/paste/etS6SItP#B0a02dge4nvuSvd08GblH6ouT2B8i1dCvFj7+50wxVX
Rules Tab Screenshot:
https://postimg.cc/rKwFQ8JT

iso-8859-1 doesn't appear to be the right encoding. It doesn't have codes to represent the characters you provided in your example (https://www.charset.org/charsets/iso-8859-1).
Not sure how it works in Python, but in Java, all Strings are internally represented as Unicode, so there isn't a need to transition between encodings while text is still in String form (unlike respS = respS .encode('iso-8859-1') which appears to be changing the internal string encoding since the output is still a string). In Java, the encoding should only come into play when converting the String to bytes or from bytes to a String. In our case, Reshaper is using the recommended stringToBytes method provided by Burp Suite rather than using Java's built-in library for encoding. It appears that the problem is in the way their method is handling the String. It does not occur using Java's built-in library. I will have to see what Burp Suite folks say about this issue.

Hi thanks for digging into this problem
Yes there are some ambiguity in charset of encoded strings. (I think it is doubly encoded or sth) or it could be stringToBytes method misinterpreting the encoding of the data.
About StringToSting conversion you are right. The data retrived from burpAPI methods, is first decoded from byte to string and then they are reassembled again into byte and sent back to burpAPI, but in non-ASCII it's somehow needed that the same STRING gets encoded again and the outcome needs the stringtobyte . Basically one extra step is required (a encoding conversion of let's say StringToString). I was trying to implement the equivalent code of Python in Java and I got "Unhandled exception type UnsupportedEncodingException" . I fixed it using throws but I realized I have to fix every single code that uses setbody method which kinda feels wrong
ReshaperForBurp-release\src\main\java\synfron\reshaper\burp\core\messages\entities\HttpResponseMessage.java I believe is the actual script we need to apply our changes and I change setBody as following:
`
import java.io.UnsupportedEncodingException;
public void setBody(String body) throws UnsupportedEncodingException {
String decodedToUTF8 = new String(body.getBytes("ISO-8859-1"), "UTF-8");
this.body = new HttpBody(TextUtils.stringToBytes(decodedToUTF8));
//this.body = new HttpBody(TextUtils.stringToBytes(body));
changed = true;
}`
When I try to compile, it fails saying "UnsupportedEncodingException" where ever setBody is used so I'm trying to feel that this is not the way of StringToString conversion. So I would appreciate it if you could help me to implement the the equivalent of respS = respS .encode('iso-8859-1') in Java. It seems much easier in Python but seemingly in Java matter of encoding  applies where we have Bytes and String form has its challenges.

I believe you miss interpreted my comment above. First, there is no need to worry about String to String encoding. Second, ISO-8859-1 would be the incorrect encoding to use as an output because it does not support all of the characters in the example text you gave.
Below is the git patch to change what Reshaper uses for String to Byte conversion to use UTF-8.
diff --git a/src/main/java/synfron/reshaper/burp/core/utils/TextUtils.java b/src/main/java/synfron/reshaper/burp/core/utils/TextUtils.java
index 37083bf..32ec8e5 100644
--- a/src/main/java/synfron/reshaper/burp/core/utils/TextUtils.java
+++ b/src/main/java/synfron/reshaper/burp/core/utils/TextUtils.java
@@ -109,7 +109,7 @@ public class TextUtils {
     }

     public static byte[] stringToBytes(String value) {
-        return BurpExtender.getCallbacks().getHelpers().stringToBytes(StringUtils.defaultString(value));
+        return StringUtils.defaultString(value).getBytes(StandardCharsets.UTF_8);
     }

     public static String bufferAppend(String text1, String text2, String suffix, int maxLength) {

Hi Thanks for your help I managed to compile it using Gradle 6.8 and Java 15 just as you mentioned. However compiling using the latest versions of Java and Gradle failed saying "No duplicate handling strategy has been set" and I know it could be fixed by configuring some settings but I just wanted to avoid complexities.
The committed cahnge works just fine and I haven't seen any problems so I'd recommend considering this Method over the previous one because it just supports everything I suppose, and as you are the developer still you can decide better.
Thank you so much you for this great help and amazing extension you developed. It is really interactive and modular.

Glad the change worked for you. I have left a message on the Burp Suite forums concerning the issue you found. Hopefully they will fix that issue so this change doesn't become necessary.

Please see release v1.6.0 (Not yet available in the BApp Store). An option to change the default encoding has been added to the Settings. A new Then action (Set Encoding) was also added for more targeted use cases.
https://github.com/synfron/ReshaperForBurp/releases/tag/v1.6.0
Thanks for reporting this issue. 🙂

Hi
Thank you so much for fixing this issue and helping me to get what I needed.
I just downloaded this new release and now I see the encoding option.  I believe I couldn't configure it properly so I would be grateful if you could tell me how to configure it to make it work in my specific case.

Looks like there is a bug in the request encoding detection (is in the wrong location).
As a workaround for now, to emulate the same general functionality create a new Rule like below at the beginning of your Rules list:
When - Event Direction - Request
When - Content Type - JSON, XML, URL Encoded
Then - Set Encoding - Auto-detect
FYI, There are some texts where auto-detect will not work well because auto-detection is requires heuristics. To get the most accurate output, I'd recommend only setting the encoding in locations to handle very specific situations (as is possible with the Then), and using a specific encoding that is known to have the expected output.

@farid220 Can you please try https://github.com/synfron/ReshaperForBurp/releases/tag/v1.6.0-next and see if it fixes your issue?

I'm so sorry I don't know why Github didn't send me notification about your message. I tested version 1.6.0 and it works just fine (except the very default Encoding setting).
I see you committed 20 hours ago and I will test this one.
I encountered sth else though and I think it might be my inability to set it up.
The thing is there is a URL of an image which basically returns the image (PNG).
Burp's interceptor shows the header with some weird characters (Which is reasonable because it's image in byte format)
Now I want to implement it in Reshaper but I just can't get it done. I copy pasted the exact data from Burp interceptor but it just didn't work.
Could you please elaborate on how can I implement this in Reshaper?

v1.6.1 is release to the BApp Store to fix the encoding issue. Please open a discussion or new issue for your image problem.


Bug:  Syntax for Namespace check invalid

Describe the bug
The recent change in version 0.5.0 to check the namespace existing fails to parse with that latest SurrealDB 1.5.3, the 'IF EXISTS' causes the statement to become syntactically invalid.
Creating a new client results in the following error response:
{"level":"fatal","error":"failed to initialize client: could not define namespace: response status is not OK","time":"2024-06-16T18:47:01+01:00","message":"Connection Error"}

This would work prior this recent change, looking at the change it seems the DEFINE NAMESPACE statement changed to include IF EXISTS, which seems to fail, testing the statement manually using Surrealist results in the following:
There was a problem with the database: Parse error: Failed to parse query at line 1 column 8 expected query to end
  |
1 | DEFINE NAMESPACE IF EXISTS test_namespace;
  |        ^ perhaps missing a semicolon on the previous statement?

Steps to reproduce
Latest version (at time of writing 0.5.0) of the sdbc module.
SurrealDB Server Version 1.5.3
	client, err := sdbc.NewClient(ctx, sdbc.Config{
		Host:      "localhost:8000",
		Username:  "root",
		Password:  "root",
		Namespace: "test_namespace",
		Database:  "test_db",
	})

	if err != nil {
		log.Fatal().Err(err).Msg("Connection Error")
	}

Expected behaviour
Valid connection and client to be initialised and returned.
SDBC version
sdbc v0.5.0
Contact Details
<EMAIL_ADDRESS>Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

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

Ignore... Something screwy happened with our test server.. sorry..


ID2 Completion

Please refer to the attached files.

rejected:ID2
reason:We cannot fully verify this submission. It looks like you are trying to submit ID2, however, it seems like there are no address submitted.

pending:ID2
account1:screenshot-TBC
account2:screenshot-TBC
We cannot fully verify this submission. Please assist us in typing out the account addresses for ID2 in full as text in a comment on this PR. Thank you!
@m-emre

Account1:
3kvPnGZRfEj1VnnrpmGFQMXBDGtoveehsbRi8TQfinHaFKiNB3
Account2:
4pRsNPKiCKrpCFg4u7w3Nuvc9UtLhCUVixEifTFuN7qpVjn3FX

Account1:
3kvPnGZRfEj1VnnrpmGFQMXBDGtoveehsbRi8TQfinHaFKiNB3
Account2:
4pRsNPKiCKrpCFg4u7w3Nuvc9UtLhCUVixEifTFuN7qpVjn3FX

confirmed:ID2
account1:3kvPnGZRfEj1VnnrpmGFQMXBDGtoveehsbRi8TQfinHaFKiNB3
account2:4pRsNPKiCKrpCFg4u7w3Nuvc9UtLhCUVixEifTFuN7qpVjn3FX
PublicDataAttributes:Verified
DeviceMake&Model:Verified

confirmed:ID2
account1:3kvPnGZRfEj1VnnrpmGFQMXBDGtoveehsbRi8TQfinHaFKiNB3
account2:4pRsNPKiCKrpCFg4u7w3Nuvc9UtLhCUVixEifTFuN7qpVjn3FX
PublicDataAttributes:Verified
DeviceMake&Model:Verified


[Logs] Fix a typo in Datadog's ingestion webhook attributes description

As per Datadog docs:

host should be hostname
dtags should be ddtags

Thanks for the PR!
We will make some fix on Logpush API, then will merge this.

It turns out thathost and hostname are both valid: https://docs.datadoghq.com/logs/processing/pipelines/#special-pipelines
The only change should be fixing dtags to be ddtags.

Thanks again for the PR, @bakhtin!


How can documentation generators introspect a webargs field?

The introductory documentation to webargs lists one framework feature as:

Automatic documentation. The metadata that webargs provides can serve as an aid for automatically generating API documentation.

Any pointers on where to access that metadata?

I have no idea what metadata the doc is talking about.
If you're interested in automatic doc generation, you should pay a look at apispec.
Adding a layer on top of marshmallow/apispec/webargs allows you to specify schemas for input/output and document them at the same time. This is what flask-apispec or flask-rest-api do.

That bit is referring to the metadata stored on fields, in the form of stored arguments. For example:

>>> f = fields.Str(required=True, description='Your username')
>>> f.required
True
>>> f.metadata
{'description': 'Your username'}

Because input schemas are separated from view functions, libraries (like the ones @lafrech mentioned) can easily introspect the schemas and generate API specifications.

That bit is referring to the metadata stored on fields

Oh, of course. I didn't think of that.


Run Intel VTune against SlowMemcpyWorkload

zref: #5

Installing Intel VTune on Ubuntu:

sudo apt install pkg-config
Install VTune using apt following these instructions.
Set all these to 0 (I used sudo emacs <filename>):
4. /proc/sys/kernel/yama/ptrace_scope
5. /proc/sys/kernel/perf_event_paranoid (from here)
6. /proc/sys/kernel/kptr_restrict
Follow Intel's post-install instructions:
8. source /opt/intel/oneapi/vtune/latest/env/vars.sh
9. vtune-self-checker.sh
vtune-gui

Running zarr-benchmark within VTune
I created a very simple shell script which activates the Python venv and runs the benchmark:
#!/bin/bash

# Activate venv:
source /home/jack/python_venvs/perfcapture/bin/activate

# Run zarr-benchmark:
/home/jack/python_venvs/perfcapture/bin/python \
    /home/jack/dev/zarr/perfcapture/scripts/cli.py \
    --data-path /home/jack/temp/perfcapture_data_path \
    --recipe-path /home/jack/dev/zarr/zarr-benchmark/recipes

What does this benchmark do?
Here's the code. It's very simple!
Results
SlowMemcpyWorkload takes 5.4 seconds to run on my machine. Here's the VTune Hotspots Summary:

VTune shows that LZ4 decompression takes the most time (not surprising). What is perhaps more surprising - and is consistent with Vincent's observations - is that the second longest running function is __memmove_avx_unaligned_erms (ERMS is a CPUID feature which means "Enhanced REP MOVSB" (source). I think clear_page_erms is this 4-line ASM function.):

("CPI rate" is "Cycles Per Instruction retired". Smaller is better. The best a modern CPU can do is about 0.25.)

In last week's meeting, we discussed the hypothesis that the fact that the code is using a memmove function with "unaligned" in the name meant that the data was unaligned in memory, and hence the system was forced to use a slow "unaligned" memmove function. The hypothesis was that we could speed things up by aligning the data in memory.
After reading more about memmove, I no longer thing this hypothesis is correct. My understanding is that these memmove_unaligned functions do spend the majority of their time moving data very efficiently using SIMD instructions. The "unaligned" word in the name just means that the function handles the "ragged ends" and the start and/or end.
Lots of good info here: https://squadrick.dev/journal/going-faster-than-memcpy.html


Docs for writing an application from scratch

Resolves RHOARDOC-1148

[x] Have you followed the guidelines in our Contributing document?
[x] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?

This is the legacy content modularised. I have tried the procedure, and it works.

Please review the changes, I'll squash the commits before merge.

Squashed. Please merge and cherry-pick to the 7.1.x branch in the prod repository.


⚠ Bofrost.de has degraded performance

In ce30229, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 930 ms

Resolved: Bofrost.de performance has improved in d4a42af after 18 minutes.


dist files not updated?

I have updated vuejs from 2.4.3 to 2.4.4 and it seems that vue.js and maybe vue.min.js did not receive any changes except version number:
diff --git a/lib/vue/js/vue.js b/lib/vue/js/vue.js
index 4451bc6..0b3bb9c 100644
--- a/lib/vue/js/vue.js
+++ b/lib/vue/js/vue.js
@@ -1,5 +1,5 @@
 /*!
- * Vue.js v2.4.3
+ * Vue.js v2.4.4
  * (c) 2014-2017 Evan You
  * Released under the MIT License.
  */
@@ -4713,7 +4713,7 @@
   }
 });

-Vue$3.version = '2.4.3';
+Vue$3.version = '2.4.4';

 /*  */

I have copied this (and minimized) file from dist folder.

Well, have it your way.


Support for generic ODBC\JDBC Connection

We are looking at whether or not we could use Azure Data Studio to query Delta Lake Tables. I realize that there is no native extension, but Delta Lake also provides a JDBC interface to query Delta Lake Tables.
Describe the solution or feature you'd like
An extension that would allow support for a generic OBDC\JDBC connector.
Describe alternatives you've considered
Right now, seems like our only option would be to use another tool like DBeaver to connect to our Delta Lake tables.

Im looking for suppor for Teradata but ODBC/JDBC could be the way too. Also here are other requests that would get resolved with this one.
https://github.com/microsoft/azuredatastudio/issues/5757
https://github.com/microsoft/azuredatastudio/issues/7473

Any progress on this request?
We have a prospect who is interested in connecting a SQL interface to one of a variety of sources. Azure Data Studio doesn't have ootb connections to all of these, and more concerning, it doesn't provide an extensible method of adding other sources.  The prospect might want to add a new source later, and I don't want them to be waiting for Microsoft to decide whether it's a worthwhile extension. This particular issue is "Backlog", 13+ months old with no activity since it was opened...
We're recommending more open and flexible alternatives (eg DBeaver, DBVisualizer).

Is there any update on this topic? To query Azure SQL database tables together with Azure Data Lake files data would be great. It would make our work much more agile if we could avoid extracting files from Azure Data Lake to SQL database and only then working with the data.

Is there any update on this topic? To query Azure SQL database tables together with Azure Data Lake files data would be great. It would make our work much more agile if we could avoid extracting files from Azure Data Lake to SQL database and only then working with the data.

@marijusha Have you looked at Dremio? It provides a fast access layer to ADLS data. It can also connnect to SQL data sources. And there's a free Community Edition version.

@marijusha Have you looked at Dremio? It provides a fast access layer to ADLS data. It can also connnect to SQL data sources. And there's a free Community Edition version.

Is this even on Microsoft's radar?  Seems like a pretty obvious way to expand the usefulness and user base of the product...


Bump setup-python to latest

This internally uses node 20 now, suppressing the warning we were getting

Superseded by #8141


fix(.env): add LOGLEVEL

See https://github.com/AEGEE/core/pull/327

@serge1peshcoff @linuxbandit not sure if you have already seen this (and referenced) PR(s), but this should reduce the logging of the modules by quite a bit


How do we use the pretrained attention aligner?

Hi, I find that getting a pretrained predictive aligner (aligner='charsiu/en_w2v2_fc_10ms') to work with librispeech seems straightforward. However, I'm unable to get the attention aligner working - how do I go about initializing the aligner and how do I get the corresponding bert config to go with it? Keeps throwing an error for the same.

What type of error did you get? Did you try our colab tutorial? There are some example scripts for loading the model.


No terminal colors with mvn

VSCode Version: 1.23.1
OS Version: Windows 10 Enterprise  v1703

There is no color highlighting when I run maven inside the vscode terminal.
I would think that this is a maven bug, but the highlighting works flawlessly when I run it from the windows or Git Bash command prompt.
Steps to Reproduce:
Running mvn in the windows command prompt or Git Bash shows the colors correctly.
Running jest also shows the colors correctly in vscode.
Running echo -e "Default \e[36mCyan" also works and shows the color correctly.
Running  mvn or mvn.cmd in the vscode terminal prints the color codes instead of showing colored text.
Output:
[?[1;34mINFO?[m] ?[1m------------------------------------------------------------------------?[m
[?[1;34mINFO?[m] ?[1;32mBUILD SUCCESS?[m
[?[1;34mINFO?[m] ?[1m------------------------------------------------------------------------?[m
[?[1;34mINFO?[m] Total time: 53.568 s
[?[1;34mINFO?[m] Finished at: 2018-05-14T16:08:50+02:00
[?[1;34mINFO?[m] ?[1m------------------------------------------------------------------------?[m

Does this issue occur when all extensions are disabled?: Yes

Duplicate https://github.com/Microsoft/vscode/issues/45693