SMART App redirect_uri does not match specification
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 74/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- react, typescript
- Domain
- authentication, frontend
Research direction
Start in the SMART Health Checks application by searching for healthchecks.smartforms.io and the SMART authorization request entry point. Verify where redirect_uri is assembled, update it to include the trailing slash required by the specification, and confirm that the authorization request sends the exact expected URI.
Written by the indexing model from the issue text.
Description
Describe the bug
The SMART Forms integration specification defines the redirect URI as:
https://healthchecks.smartforms.io/
However, the SMART app sends the redirect URI without the trailing slash:
https://healthchecks.smartforms.io
This causes a redirect URI mismatch when performing strict URI validation.
To Reproduce
- Launch the SMART Health Checks application.
- Inspect the SMART authorization request.
- Observe the
redirect_urivalue sent by the application. - Compare it with the redirect URI defined in the SMART Forms integration specification.
Expected behavior
The SMART app should send the redirect URI exactly as defined in the specification:
https://healthchecks.smartforms.io/
- Dominant language
- TypeScript
- Stars
- 61
- Forks
- 29
- Avg merge
- 2d 18h
- Merged PRs (30d)
- 19
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from aehrc/smart-forms
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
aehrc/smart-forms#2122 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
aehrc/smart-forms#2121 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
aehrc/smart-forms#2110 ·
-
Difficulty 3/5 1-2 days Newbie friendliness 78/100
aehrc/smart-forms#2108 · 1 assignee ·
-
in progress
Difficulty 5/5 Over a week Newbie friendliness 35/100
aehrc/smart-forms#2089 · 2 comments · 8 reactions ·
All issues in aehrc/smart-forms
Similar issues
-
calcite-components needs triage refactor
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Esri/calcite-design-system#15203 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 91/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Automattic/studio#4908 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100