Dev #1
Open
Dev #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…time dependencies
Inspector/Tests: Added support to read actual SAZ archives and use these in the tests
…rize endpoint request
…e build dependencies
…ntation Inspector: Added temporarily WebFormEditor control to render grid views (borrowed it from Fiddler)
… coverage Inspector: Factored out session loading into SessionLoader class
…yment customizations
…ens in the console
…d existing ones to keep naming schema consistent
…ve client QA: Cummulative refactoring for both request/response inspectors tests QA: Added some new test samples for new types of request/response supported
… test sample to cover an incoming auth code response scenario
…a POST) Inspector: Refactored commonly used functionality between response/request inspectors into 'OidcInspector' base class QA: Added test coverage for incoming auth code response (via POST)
…o expand visible/scrollable area and aid usability
…t sample to cover token endpoint scenario
…point (+ some test coverage)
…'build.cmd' script
…ct's tests in the console
…:' character in 'jwt-decode.ps1' script
… claims to look like this -> 1428036539 (4/2/2015 9:48:59 PM)
|
Of course interested! 👍 Let me get acquainted with the changes and I will merge these back. |
… more input elements in the body.
…horization code response. This allows IdentityServer3 signin responses to be viewed.
|
Thanks for tweaking the inspector to enable support of IdentityServer! Could you please address comments left for some of the commits so I can proceed with merging your work to the repo? Appreciate! |
|
It would be great to have your changes merged into the main tree, support for IdentityServer is a very good stuff to have in... 👍 Let me know if I can be of any help to make your pull request as easy as possible. |
|
Also just realized the target branch chosen to merge your pull request should be dev instead of master. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi
Not sure if you're interested, but I've modified the filtering, removing /oauth2, to enable non-standard endpoint URLs such as those created by https://github.com/IdentityServer/IdentityServer3 implementations.
Cheers
Dave