Skip to content

Soft fixes for broken interfaces#40

Merged
197g merged 3 commits intomasterfrom
fix-interfaces
Aug 9, 2019
Merged

Soft fixes for broken interfaces#40
197g merged 3 commits intomasterfrom
fix-interfaces

Conversation

@197g
Copy link
Owner

@197g 197g commented Aug 8, 2019

Fixes the interface issues discovered in #39.

Deprecates the broken interface, they will be removed in the next major version. We avoid a breaking change in the case of private extension iteration by introducing a boolean flag to turn the public extension iterator into the private one. Renames the constructors for both iterators to a more 'rusty' version which is more similar to HashMap::keys and other specialized container iterators.

This fixes a bug where the Assertion grant type could silently discard private extensions instead of returning with an error.

197g added 2 commits August 8, 2019 13:50
Old one will be removed with the next breaking release cycle.
Avoid the breaking change, for now, by simply deprecating the methods
and introducing an internal bool switch in the public iterator. This
hack will be removed in the next major version.
@197g
Copy link
Owner Author

197g commented Aug 9, 2019

This fails due to nightly breakage, that would only affect testability of a dependency of iron, not testability of oxide-auth. See this PR that is available to fix it.

@197g 197g merged commit b0ae64c into master Aug 9, 2019
@197g 197g deleted the fix-interfaces branch August 9, 2019 00:01
197g added a commit that referenced this pull request Aug 17, 2019
Soft fixes for broken interfaces
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.

1 participant