Skip to content

Commit

Permalink
RESSUP-1301: Remove the protocol and host from the redirect url
Browse files Browse the repository at this point in the history
  • Loading branch information
iambrandonn committed Dec 22, 2015
1 parent 6b4bd29 commit 54e2d57
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
V1.0.3
* Fixed an issue when running the mock auth client that used the incorrect host
in some situations. Fixed by no longer including protocol of host on the
redirect url.
* Progress through a disclosure is now tracked.
If a user gets halfway through and stops, when they return
they will be brought back to where they were. The data has always been
Expand Down
2 changes: 1 addition & 1 deletion server/services/AuthService/mockAuthClient.js
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ export function getUserInfo(dbInfo, hostname, token) {
}

export function getAuthLink(req) {
const returnToValue = encodeURIComponent(`${req.protocol}://${req.hostname}${req.originalUrl}`);
const returnToValue = encodeURIComponent(req.originalUrl);
return `/coi/auth?return_to=${returnToValue}`;
}

0 comments on commit 54e2d57

Please sign in to comment.