Doplnění dokumentace Keychain proxy do developerské dokumentace#30
Doplnění dokumentace Keychain proxy do developerské dokumentace#30MatejHamala wants to merge 2 commits intogh-pagesfrom
Conversation
|
|
||
| When your app receives a proxied request, it **must verify the JWT token** before processing the request: | ||
|
|
||
| 1. **Check expiration** — reject tokens where `expires_at` is in the past. |
There was a problem hiding this comment.
Možná by bylo vhodné zmínit v jaké formě ta expires_at vlastně bude.
Obecně, jak nad tím tak přemýšlím, by možná bylo nejlepší se vyvaroval toho, aby se expires_at prezentovalo jako datetime, a místo toho jej možná spíš udávat jako timestamp <kdy expiruje>.
U datetimes se musí (měly by se) řešit timezone - a podle formátu tam buď TZ vůbec třeba (pak se neví, jestli je to UTC, nebo třeba naše evropsko-české letní +2 a zimní +1 - a nedá se to jednoduše odvodit, protože nevíš, jestli to má být UTC, nebo nějaké jiné, nebo tam jen informace o TZ chybí - a to je bordel). Nebo tam ten TZ být může a pak je na programátorech a nebo použitých knihovnách, jestli a jak pak takové datetimes+TZ správně vyhodnocují a porovnávají.
Myslím, že s timezones programátoři "rádi a často" dělají chyby, takže nejlepší by bylo tu variantu úplně eliminovat. Nemít a neřešit timezones, tzn. nemít a neřešit datetimes - a mít políčko expires_at prostě jako UNIX timestamp. A mít tu ukázaný příklad.
Close: https://github.com/mergado/mergado/issues/14069#issuecomment-4003218914
Includes: