Skip to content

Commit 88d3a86

Browse files
authored
Merge pull request #1765 from nodeSolidServer/fix/issue#1764
check for redirect_uri
2 parents bd171fa + 1deb9db commit 88d3a86

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

lib/requests/sharing-request.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,11 @@ class SharingRequest extends AuthRequest {
6464
* @param req {IncomingRequest}
6565
* @param res {ServerResponse}
6666
*/
67-
static async get (req, res) {
67+
static async get (req, res, next) {
6868
const request = SharingRequest.fromParams(req, res)
6969

7070
const appUrl = request.getAppUrl()
71+
if (!appUrl) return next()
7172
const appOrigin = appUrl.origin
7273
const serverUrl = new url.URL(req.app.locals.ldp.serverUri)
7374

@@ -153,6 +154,7 @@ class SharingRequest extends AuthRequest {
153154
}
154155

155156
getAppUrl () {
157+
if (!this.authQueryParams.redirect_uri) return
156158
return new url.URL(this.authQueryParams.redirect_uri)
157159
}
158160

0 commit comments

Comments
 (0)