Skip to content

Replace url of Star Wars API#10

Merged
sonylnagale merged 1 commit intoPacktPublishing:masterfrom
LuRsT:fix-swapiurl
Jun 19, 2020
Merged

Replace url of Star Wars API#10
sonylnagale merged 1 commit intoPacktPublishing:masterfrom
LuRsT:fix-swapiurl

Conversation

@LuRsT
Copy link
Contributor

@LuRsT LuRsT commented May 27, 2020

The url for the SWAPI has changed from swapi.co to swapi.dev

More here

The url for the SWAPI has changed from `swapi.co` to `swapi.dev`
@sonylnagale
Copy link
Collaborator

@LuRsT has this propagated? The cert is invalid and bypassing that with Chromium just says "Unknown app".

@LuRsT
Copy link
Contributor Author

LuRsT commented May 28, 2020

@sonylnagale, yeah, it has propagated, at least on my end, swapi.co doesn't even exist anymore and swapi.dev seems to have the right certificates. I get unknown app with swapi.co.

@sonylnagale
Copy link
Collaborator

@LuRsT interesting—on my end I see that Puma-dev CA is not trusted and Chrome refuses to display.

@LuRsT
Copy link
Contributor Author

LuRsT commented May 28, 2020

Odd, could it be your DNS supplier? The certificate is from Let's Encrypt. Here is the screenshot of the about page from swapi:

2020-05-28-182557_1268x506_scrot

I guess the issue here is that @phalt stopped maintaining swapi more here, so that's @juriy 's deployment.

@sonylnagale
Copy link
Collaborator

@LuRsT it seems that macOS doesn't trust the Puma-Dev cert (at least, not by default) so I suppose the only option would be to file an issue. I'll do that and thank you very much for pointing this out.

@sonylnagale
Copy link
Collaborator

sonylnagale commented Jun 14, 2020

@LuRsT I finally figured out the problem: I had a DNS issue of all things, so it routed the SSL incorrectly. I should have checked DNS when you suggested. :) When I use Google's DNS servers, it's fine. I'll go ahead apply these changes after a quick check. Thanks!

@sonylnagale sonylnagale merged commit cce3745 into PacktPublishing:master Jun 19, 2020
@LuRsT LuRsT deleted the fix-swapiurl branch June 22, 2020 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants