Skip to content

warden: firewal.Audience overridden with requesting clients subject#236

Merged
aeneasr merged 1 commit intoory:masterfrom
mfzl:master
Aug 27, 2016
Merged

warden: firewal.Audience overridden with requesting clients subject#236
aeneasr merged 1 commit intoory:masterfrom
mfzl:master

Conversation

@mfzl
Copy link
Copy Markdown
Contributor

@mfzl mfzl commented Aug 27, 2016

firewal.Context.Audience overridden with requesting clients subject in TokenAllowed and TokenValid

Fix #237

…n TokenAllowed and TokenValid

Signed-off-by: Mohamedh Fazal <mohamedhfazal@gmail.com>
@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Aug 27, 2016

Thank you for the PR but I remember now what this was for. I will include your fix in another fix. :)

@aeneasr aeneasr added this to the 0.4.2 milestone Aug 27, 2016
@aeneasr aeneasr added the bug Something is not working. label Aug 27, 2016
@aeneasr aeneasr self-assigned this Aug 27, 2016
@mfzl
Copy link
Copy Markdown
Contributor Author

mfzl commented Aug 27, 2016

No problem

On Sat, Aug 27, 2016, 19:28 Aeneas notifications@github.com wrote:

Thank you for the PR but I remember now what this was for. I will include
your fix in another fix. :)


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#236 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABs0pmWzR2K4UPHQzYAwNcJSOk7TUd78ks5qkEl1gaJpZM4Jur7Y
.

@aeneasr
Copy link
Copy Markdown
Member

aeneasr commented Aug 27, 2016

ah no nvm :) thanks!

@aeneasr aeneasr merged commit d5c267f into ory:master Aug 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is not working.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants