Hey @GMOTGIT 👋🏼! Thanks a lot for your contribution, it's very welcomed!

The changes you've done would eventually work, since you've replaced the return with a throw, and we handle that when we call the createRemoteLink function. However, the place where you've done this change is not the best, since you've updated the build artifact of the main source code, which will get replaced if do any change to the original source code, the index.js file.

I suggest you to handle this in a slightly different way so that your changes don't get overwritten in the future.

Since what we want to achieve is to check if the HTTP call we're doing returns an authentication/authorization error (which in HTTP terms means the status code of the response will be either 401 or 403, respectively), a more "bullet-proof" approach would be to check if the response we're receiving has either of those status codes.

If you look closely, we're using the Fetch API to perform the HTTP request. The return of that function is of type Response, and includes a property called status, which is a number representing the status code of the response.

We could use that to easily check and validate our request was successfully completed and didn't had any authentication error.

On L#44 when we do the request, instead of returning immediately the result of that call, we could instead put it in a variable, validate the status code, throw an error in case the call wasn't successful or return the response as-is and let the program continue on.

Something like this:

const resp = await fetch(...);

if (resp.status === 401) {
  throw new Error("unauthenticated");
}

if (resp.status === 403) {
  throw new Error("unauthorized");
}

return resp;

Please, let me know if this helps you or if you've any further questions. We're happy to on-board you and help you however you need! GJ 👍🏼