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

Fix/sanitize err #74

Merged
merged 6 commits into from
Jul 28, 2023
Merged

Fix/sanitize err #74

merged 6 commits into from
Jul 28, 2023

Conversation

gwnlng
Copy link
Contributor

@gwnlng gwnlng commented Jul 28, 2023

  • Tests written and linted ℹ︎
  • Documentation written ℹ︎
  • Commit history is tidy ℹ︎

What this does

This PR sanitize the Axios error config header Authorization property from leaking sensitive data into logs.

Notes for the reviewer

Check jest result on call to /gotimeout testcase to assert masked headers Authorization property

More information

gwnlng added 6 commits July 27, 2023 14:00
moved ci to github actions
included response interceptor to sanitize error
fixed dependencies package
run build-test on push of all branches
prettier formated code style
set snyk monitor org
removed circleci workflow
included timeout testcase on v1 api endpoint
@gwnlng gwnlng requested a review from a team as a code owner July 28, 2023 08:34
@gwnlng gwnlng merged commit ab4244e into develop Jul 28, 2023
@gwnlng gwnlng deleted the fix/sanitize-err branch July 28, 2023 14:04
@github-actions
Copy link

github-actions bot commented Oct 5, 2023

🎉 This PR is included in version 1.8.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants