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

Add missing encodeURIComponent on username password reset #6278

Merged
merged 2 commits into from
Dec 17, 2019

Conversation

Klaitos
Copy link
Contributor

@Klaitos Klaitos commented Dec 13, 2019

#6277
Here you go @davimacedo

@davimacedo
Copy link
Member

Thanks @Klaitos . Can you please add a test case as well?

@codecov
Copy link

codecov bot commented Dec 15, 2019

Codecov Report

Merging #6278 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6278      +/-   ##
==========================================
- Coverage   93.85%   93.84%   -0.01%     
==========================================
  Files         169      169              
  Lines       11564    11566       +2     
==========================================
+ Hits        10853    10854       +1     
- Misses        711      712       +1
Impacted Files Coverage Δ
src/Routers/PublicAPIRouter.js 92.3% <100%> (+0.13%) ⬆️
src/RestWrite.js 93.75% <0%> (-0.17%) ⬇️
src/GraphQL/loaders/parseClassTypes.js 94.23% <0%> (ø) ⬆️
...dapters/Storage/Postgres/PostgresStorageAdapter.js 97.01% <0%> (ø) ⬆️

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 e0e06ef...3e491ab. Read the comment docs.

@Klaitos
Copy link
Contributor Author

Klaitos commented Dec 16, 2019

Thanks @Klaitos . Can you please add a test case as well?

Done !

Copy link
Member

@dplewis dplewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test looks good! Thanks for contributing.

@dplewis dplewis merged commit d9f36b0 into parse-community:master Dec 17, 2019
UnderratedDev pushed a commit to UnderratedDev/parse-server that referenced this pull request Mar 21, 2020
…unity#6278)

* Add missing encodeURIComponent on username

* Add new unit test on encoded username password reset redirect
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.

3 participants