ElevatoDigital / dewdrop

Dewdrop makes writing complex WordPress plugins simpler by providing a sensible project layout and developer tools.
Other
11 stars 3 forks source link

Add HTTPS support for reset password email link #87

Closed bravadomizzou closed 8 years ago

bravadomizzou commented 8 years ago

The link you get (in the email) when you reset your password looks like this:

http://demo.thegraduationcenter.com:443/ and the browser says it is a bad request

darbyfelton commented 8 years ago

Good catch! I had added HTTP support for other ports than 80 to handle my host port mapping to my Vagrant instance, but I didn't add support for HTTPS at the same time.

Here is the relevant code: https://github.com/DeltaSystems/dewdrop/blob/master/Dewdrop/Auth.php#L203

darbyfelton commented 8 years ago

I believe this is resolved with https://github.com/DeltaSystems/dewdrop/releases/tag/1.23.0