Consolidate BrowserAuthCallbackHandler and BrowserAuthorizationClient

[ben-polinsky]

Merge BrowserAuthCallbackHandler into BrowserAuthorizationClient

Fixes: #103

[ben-polinsky]

I've tested this out using the integration tests in #119 and they pass, FYI

[aruniverse]

thx for killing off the static

[aruniverse]

I've tested this out using the integration tests in #119 and they pass, FYI

lets get that pr merged in before this

[aruniverse]

@WillGrayMSU did you pull this into DR and test in a CoSpace to verify this worked fine?

[WillGrayMSU]

@WillGrayMSU did you pull this into DR and test in a CoSpace to verify this worked fine?

I haven't, but I can try now

[WillGrayMSU]

types need to be exported

[ben-polinsky]

@aruniverse I'm unable to test the cra-template due to some annoying pre-release/peer deps constraints. Perhaps we should publish this as a pre-release or alpha to test.

[aruniverse]

getting a pre-release out will be annoying, since the change is marked as minor and its currently 0.x, no one should be pulling this in automatically. i guess we can test once released

[ben-polinsky]

getting a pre-release out will be annoying, since the change is marked as minor and its currently 0.x, no one should be pulling this in automatically. i guess we can test once released

Maybe it should be major... it's a breaking change

[aruniverse]

getting a pre-release out will be annoying, since the change is marked as minor and its currently 0.x, no one should be pulling this in automatically. i guess we can test once released

Maybe it should be major... it's a breaking change

according to semver, breaking changes are allowed in 0.x

[ben-polinsky]

getting a pre-release out will be annoying, since the change is marked as minor and its currently 0.x, no one should be pulling this in automatically. i guess we can test once released

Maybe it should be major... it's a breaking change

according to semver, breaking changes are allowed in 0.x

the more you know! Thanks