Bring over ValidationHelper functionality from v4 branch

[golang-jwt]

Bring over ValidationHelper functionality from v4 branch

[oxisto]

Just ran into the "token used before issued" error today for the first time, after things had been working fine previously.

The discussion at #98 referenced this issue. Are there any plans to implement it and disregard the pre-IAT check?

In a world with infinite time, yes :) Currently I am waiting for #108 to be merged. This one will be next.

[oxisto]

I had a quick look yesterday and I am not sure if we can port this over in a backwards compatible manner, since the interface of Claims needs to be changed. Especially the Valid function needs to have an additional parameter for the ValidationHelper. I guess I will first focus on the functionality and then see whether we can port this to v4 or if this will then introduce a v5 branch with breaking changes.

[mcquackers]

I was wondering about the implementation of the leeway:
Why does the incorporation of leeway depend on the validating server instead of the authorizing client?
To me, it seems like it would be cleaner for the JWT to be the source of truth, with leeway built into the iat/exp

[dgurns]

Just ran into the "token used before issued" error today for the first time, after things had been working fine previously.

The discussion at #98 referenced this issue. Are there any plans to implement it and disregard the pre-IAT check?

[dgurns]

Awesome, appreciate your work!

[miparnisari]

Currently I am waiting for #108 to be merged.

it's been merged 👀 :)

[ksegun]

This is something we relied on from the old library as we ran on the preview version and would help us move to adopt this port.

[arinto]

Yes, we also rely in this functionality from the old library :)

@oxisto , regarding your statement below:

I guess I will first focus on the functionality and then see whether we can port this to v4 or if this will then introduce a v5 branch with breaking changes.

So, what are the intended functionality that are targeted in v4.x?

[oxisto]

Please see #211 for an ongoing discussion how to proceed.

[oxisto]

Fixed by #234