8000 Update release notes URL for change log by binury · Pull Request #686 · feathersjs/feathers · GitHub
[go: up one dir, main page]

Skip to content

Update release notes URL for change log#686

Closed
binury wants to merge 1 commit intofeathersjs:masterfrom
binury:patch-1
Closed

Update release notes URL for change log#686
binury wants to merge 1 commit intofeathersjs:masterfrom
binury:patch-1

Conversation

@binury
Copy link
Contributor
@binury binury commented Oct 4, 2017

Fixes #665

@daffl daffl mentioned this pull request Oct 20, 2017
@daffl
Copy link
Member
daffl commented Oct 20, 2017

Thank you @teddy-error! I added a complete generated changelog instead in #695 since it was actually completely missing.

@daffl daffl closed this in #695 Oct 20, 2017
daffl pushed a commit that referenced this pull request Aug 29, 2018
As discussed in #684, the current default for subject causes token
verification to fail if sub is overriden.

Removing subject from default options might be a breaking change as
people might rely on it.

Remove subject from verification options, so it's not checked. This is
a safer way.
daffl pushed a commit that referenced this pull request Aug 29, 2018
As discussed in #684, the current default for subject causes token
verification to fail if sub is overriden.

Removing subject from default options might be a breaking change as
people might rely on it.

Remove subject from verification options, so it's not checked. This is
a safer way.
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

0