Add Plug.Conn.Status.valid?/1
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia in lib/plug/conn/status.ex, soprattutto da Plug.Conn.Status.code/1 e reason_phrase/1. Determina come gli status integrati e personalizzati debbano definire la validità, quindi conferma che il comportamento proposto di valid?/1 e la gestione degli interi siano coerenti con l’API degli status esistente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I’m working on a library that accepts status codes (integer or atom) and want to verify that valid status values are used.
I can do this with:
try
Plug.Conn.Status.code(value)
true
rescue
_ -> false
end
But this seems like it might be more generally useful if built into Plug.Status. Would a PR for this feature be accepted?
As well, it seems that Plug.Conn.Status.code/1 for the integer case isn’t entirely correct given the error raised with an unknown value in reason_phrase/1:
def code(integer) when integer in 100..999 do
integer
end
…
That feels like it should be instead
```elixir
@valid_status_codes Map.keys(Map.merge(statuses, custom_statuses))
def code(integer) when integer in @valid_status_codes do
integer
end
- Lingua principale
- Elixir
- Stelle
- 3k
- Fork
- 609
- Merge medio
- 35m
- PR unite (30g)
- 6
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di elixir-plug/plug
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
elixir-plug/plug#1299 ·
-
Kind:Bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
elixir-plug/plug#757 · 8 commenti ·
Tutte le issue di elixir-plug/plug
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
carverauto/serviceradar#4596 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
agentjido/jido_harness#80 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
sevenseacat/cinder#235 ·