Skip to content

Handle possibility of relative redirect URLs#280

Open
rogerzxu wants to merge 1 commit into
microsoft:masterfrom
rogerzxu:patch-1
Open

Handle possibility of relative redirect URLs#280
rogerzxu wants to merge 1 commit into
microsoft:masterfrom
rogerzxu:patch-1

Conversation

@rogerzxu
Copy link
Copy Markdown

Fixing issue mentioned in #201

Using the new URL constructor handles all cases correctly:

new URL('/relative', 'https://localhost:80/');
new URL('/relative', 'https://localhost:80/api);
new URL('https://localhost:80/relative', 'https://localhost:80/api);

// all become https://localhost:80/relative

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.

2 participants