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

sync fork with upstream #11

Merged
merged 220 commits into from
May 13, 2024
Merged

sync fork with upstream #11

merged 220 commits into from
May 13, 2024

Conversation

dagbay-rh
Copy link

@dagbay-rh dagbay-rh commented May 2, 2024

pr_check

I added a pr_check.sh to build our images in prs to make sure they build ok
linking file here since the diff is so huge: https://github.com/RedHatInsights/APIcast/pull/11/files#diff-46233f36c38a197d402638dc37fffcc351a00cd8d359b9de2c3923ae63008c4c

Testing

  • build locally
  • build 3scale locally with the new apicast image
  • run 3scale tests
  • run 3scale locally

eguzki and others added 30 commits February 17, 2023 10:29
…ssuer-verification

THREESCALE-9009 fix OIDC jwt key verification
THREESCALE-7735 THREESCALE-8865 opentelemetry instrumentation
tkan145 and others added 15 commits April 3, 2024 10:17
Currently, when key is base64 encoded or contains special chars, the
batcher policy will save it to the cache but won't be able to retrieve
it due to the regrex mismatch. This leads to two problems:
* Reports are not sent to the backend
* Report cache filling up over time.

This commit changes the regular expression key to accept any character
except spaces.
Previously when we changed penlight version we also removed the code
to silent writing to global variables warning which was added in
3scale@d8bfb0c

This commit has re-added the code to disable the warning again.
…olicy-storage

[THREESCALE-9537] Configure batcher policy storage
…regrex

[THREESCALE-10934] [3scale_batcher] Update regex to match key with special chars
[3scale_batcher] Update regrex to match app_id with special characters
@dagbay-rh dagbay-rh merged commit 856198e into master May 13, 2024
4 of 5 checks passed
@dagbay-rh dagbay-rh deleted the sync_upstream branch May 13, 2024 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants