Skip to content
This repository was archived by the owner on Jul 14, 2022. It is now read-only.

Remove first / at baseURL #82

Open
leonanluppi wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Remove first / at baseURL #82

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
199b050
init
leonanluppi Apr 27, 2018
1f15616
Remove first / at baseURL
leonanluppi Apr 27, 2018
c8d6646
change const to let
leonanluppi Apr 27, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion routes/shopifyAuth.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,11 @@ module.exports = function createShopifyAuthRoutes({
return {
// This function initializes the Shopify OAuth Process
auth(request, response) {
const { query, baseUrl } = request;
let { query, baseUrl } = request;
const { shop } = query;

//Remove first / at baseURL
baseUrl = baseUrl.replace(/^\//, '');

if (shop == null) {
return response.status(400).send('Expected a shop query parameter');
Expand Down