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.

Sign up for GitHub

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

Jump to bottom

Feat query shortener update #15813

Closed
wants to merge 25 commits into from
Closed

Feat query shortener update #15813

wants to merge 25 commits into from

Conversation

jason-p-pickering
Copy link
Contributor

No description provided.

amcgee and others added 25 commits March 3, 2023 02:22
@amcgee
feat: add support for shortening of long queries
f8176f1
@amcgee
chore: remove unused handler params
0137e69
@amcgee
chore: use correct NotFoundException import
0cdcfd9
@amcgee
chore: remove unused variable
c1161f4
@amcgee
Merge remote-tracking branch 'origin/master' into feat-query-shortener
9d8ff0d
@amcgee
fix: accept and produce JSON, rename to query alias, add alias/redire…
fbe04fa 
…ct endpoint
@amcgee
Merge remote-tracking branch 'origin/master' into feat-query-shortener
abe369d
@amcgee
fix: remove unnecessary cross-origin decorator
c44940e
@amcgee
fix: check validity of target property, use CodecUtils to create hash…
d1a4d96 
… alias
@amcgee
chore: rename private variable in cache region to match public name
bd6a0aa
@amcgee
chore: add Spring controller integration test
0674080
@amcgee
chore: add sha1 CodecUtils test
3eebcbf
@amcgee
feat: add more details to alias creation response including href, upd…
8c9f5df 
…ate tests
@amcgee
chore: fix code smells
c0a15ba
@amcgee
chore: deduplicate explicit subtyping
296fd78
@amcgee
fix: enforce max target length, use local QueryAlias class in QueryCo…
bc7c40c 
…ntroller
@amcgee
chore: update tests, don't test alias forwarding due to cache limitat…
699a90c 
…ions
@amcgee
fix: expire cache entries after three hours
58f6e1c
@amcgee
chore: make inner class static
c20e10a
@amcgee
Merge branch 'master' into feat-query-shortener
7dca49b
@amcgee
chore: avoid constants in test definitions
3724f98
@amcgee
Merge branch 'feat-query-shortener' of https://github.com/dhis2/dhis2…
33a4da7 
…-core into feat-query-shortener
@jason-p-pickering
Merge branch 'master' of github.com:dhis2/dhis2-core into feat-query-…
78f2b3c 
…shortener
@jason-p-pickering
Fix merge issue
bffbac3
@jason-p-pickering
Linting
640b3ed
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Dec 1, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@jason-p-pickering jason-p-pickering marked this pull request as draft December 1, 2023 09:30
@jason-p-pickering jason-p-pickering deleted the feat-query-shortener-update branch December 12, 2023 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jason-p-pickering @amcgee