Skip to content

Doplnění dokumentace Keychain proxy do developerské dokumentace#30

Open
MatejHamala wants to merge 2 commits intogh-pagesfrom
keychain
Open

Doplnění dokumentace Keychain proxy do developerské dokumentace#30
MatejHamala wants to merge 2 commits intogh-pagesfrom
keychain

Conversation

@MatejHamala
Copy link
Collaborator

Close: https://github.com/mergado/mergado/issues/14069#issuecomment-4003218914

Includes:

  • New API docs url and JWT verification paragraph
  • New post about keychain & related proxies

@MatejHamala MatejHamala requested a review from smuuf March 5, 2026 08:46

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.
Copy link
Member

@smuuf smuuf Mar 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants