Skip to content

check for the right algorithm after negotiation #32

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

check for the right algorithm after negotiation #32

wants to merge 1 commit into from

Conversation

hex7c0
Copy link

@hex7c0 hex7c0 commented Jan 27, 2015

WIP for #27

@hex7c0
check for the right algorithm
cf4cd86 
remove ternary operator and use if...else
@dougwilson dougwilson added the pr label Jan 27, 2015
@dougwilson dougwilson self-assigned this Jan 27, 2015
@nickdesaulniers nickdesaulniers mentioned this pull request Oct 27, 2015
Open
@dougwilson dougwilson force-pushed the master branch 3 times, most recently from d7bb81b to cd957aa Compare May 30, 2018 04:09
@bjohansebas
Copy link
Member

Thanks for the PR.

This is unnecessary - negotiator only returns the encodings we explicitly support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dougwilson dougwilson

Labels
needs rebase needs tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hex7c0 @bjohansebas @dougwilson