Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sanitize ULRs before parsing query params #1592

Merged
merged 2 commits into from
Oct 26, 2022
Merged

sanitize ULRs before parsing query params #1592

merged 2 commits into from
Oct 26, 2022

Conversation

akphi
Copy link
Contributor

@akphi akphi commented Oct 26, 2022

Summary

How did you test this change?

  • Test(s) added
  • Manual testing (please provide screenshots/recordings)
  • No testing (please provide an explanation)

@changeset-bot
Copy link

changeset-bot bot commented Oct 26, 2022

🦋 Changeset detected

Latest commit: 610016c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 31 packages
Name Type
@finos/legend-shared Minor
@finos/legend-application-query Patch
@finos/legend-extension-dsl-data-space Patch
@finos/legend-application-query-deployment Patch
@finos/legend-application-studio-deployment Patch
@finos/legend-application-taxonomy-deployment Patch
@finos/legend-application-query-bootstrap Patch
@finos/legend-application-studio-bootstrap Patch
@finos/legend-application-studio Patch
@finos/legend-application-taxonomy-bootstrap Patch
@finos/legend-application-taxonomy Patch
@finos/legend-application Patch
@finos/legend-art Patch
@finos/legend-extension-dsl-diagram Patch
@finos/legend-extension-dsl-mastery Patch
@finos/legend-extension-dsl-persistence-cloud Patch
@finos/legend-extension-dsl-persistence Patch
@finos/legend-extension-dsl-service Patch
@finos/legend-extension-dsl-text Patch
@finos/legend-extension-format-json-schema Patch
@finos/legend-extension-format-morphir Patch
@finos/legend-extension-store-flat-data Patch
@finos/legend-extension-store-relational Patch
@finos/legend-extension-store-service-store Patch
@finos/legend-extension-tracer-zipkin Patch
@finos/legend-graph Patch
@finos/legend-manual-tests Patch
@finos/legend-query-builder Patch
@finos/legend-server-depot Patch
@finos/legend-server-sdlc Patch
@finos/legend-storage Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link

codecov bot commented Oct 26, 2022

Codecov Report

Merging #1592 (610016c) into master (6689b21) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1592   +/-   ##
=======================================
  Coverage   42.12%   42.12%           
=======================================
  Files        1315     1315           
  Lines       57720    57721    +1     
  Branches    13049    13049           
=======================================
+ Hits        24314    24315    +1     
  Misses      33312    33312           
  Partials       94       94           
Impacted Files Coverage Δ
...d-application-query/src/components/QueryEditor.tsx 6.87% <ø> (ø)
...nd-application-query/src/components/QuerySetup.tsx 6.31% <ø> (ø)
...ace/src/components/query/DataSpaceQueryCreator.tsx 0.00% <ø> (ø)
packages/legend-shared/src/network/NetworkUtils.ts 27.62% <100.00%> (+0.40%) ⬆️

@akphi akphi merged commit 6f8aa5c into finos:master Oct 26, 2022
@akphi akphi deleted the nolan branch October 26, 2022 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-present CLA Signed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants