prestodb/presto

Incorrect HTTP response code 500 is returned when the password is too long

Aperta

#25.679 aperta il 4 ago 2025

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)Java (5240 fork)batch import
buggood first issue

Metriche repository

Star
 (15.558 stelle)
Metriche merge PR
 (Merge medio 34g 14h) (120 PR mergiate in 30 g)

Descrizione

When file-based authentication is enabled, and the password is too long, the server responds with HTTP 500, not the expected HTTP 401

Your Environment

  • Presto version used: 0.294-SNAPSHOT
  • Storage (HDFS/S3/GCS..):
  • Data source and connector used:
  • Deployment (Cloud or On-prem): on-prem with Docker Swarm
  • Pastebin link to the complete debug logs: https://pastebin.com/SvU1KCpQ

Expected Behavior

The server should return HTTP 401.

Current Behavior

The server should return HTTP 401.

Possible Solution

Catch the RuntimeException and wrap it into a PrestoException.

Steps to Reproduce

Screenshots (if appropriate)

Context

Guida contributor

Direzione di ricerca
Il problema riguarda l'autenticazione basata su file che restituisce 500 invece di 401. La correzione prevede di catturare RuntimeException e incapsularla in una PrestoException. Esamina le classi relative all'autenticazione in presto main/src/main/java/com/facebook/presto/server/security/ o simili. La classe PrestoException si trova in presto spi. I commenti nell'issue suggeriscono questo approccio. Assicurati che la correzione restituisca 401 per una lunghezza password errata anziché 500.
Tech stack
java
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJava
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75