Roderick van Domburg
Roderick van Domburg
Thanks for the update. I think there's something up with the changelog merge. Could you move your changes up to the "Unreleased" section? Also probably should mark the `discovery` changes...
Merged, thanks a lot! May I ask you to also update the wiki with the new command line options? With these breaking changes, we're going to up the version to...
So it’s saying there’s something up with the cerificate. Do you know what that could be, is there indeed tampering, wrong certificates, anything like that?
Yes, let's make it a separate issue.
The payload numbers are ASCII codes. Maybe they give some idea what’s going on.
I'm not sure I completely follow. Can you post something like a Gist with two full working examples?
> I don't want to keep this issue pending so I added a PR that fixes the issue for me. Providing two similar `get_token()` functions is maybe not the smartest...
Props for the productive conversation. Though it's not the end of it, through it all it does seem it'd be smart to have both functions. `get_token` just being a convenience...
Didn't mean that as a statement to kill the conversation, so if you think differently do not hesitate to put it on the table.
Merged, thanks!