finatra

finatra copied to clipboard

**Describe the bug** PathPattern calls QueryStringDecoder.decodeComponent, which replaces '+' to ' '. This behavior is expected in query strings, but not in path strings. (FYI, QueryStringDecoder.decodeComponent has an overloaded version...

finalchild

How to achieve graceful server shutdown for environments that rely on healthcheck to mark nodes unhealthy

2

## Background Context I'm working in a hosting environment that at a high level works like this: - Load balancer monitors `/health` endpoint, exposed via admin port, to identify if...

niodice

Restricting/removing admin routes

1

Is seems impossible to remove/restrict the access to some admin routes. I think about ressource like registry.json that we don't want everyone to access. In that case we must desactivate...

Kyochi

MDC and Log4J2 do not get along very well

1

**Describe the bug** I cannot output MDC content in logs if the log backend is `log4j2`. **To Reproduce** ```scala object MyAppMain extends MyApp class MyApp extends HttpServer with Logging {...

brisssou

fix commons-fileupload dependency for high CVEs

1

Update commons-fileupload to 1.5 to address CVE-2023-24998. This PR upgrades commons-fileupload from version 1.4 to 1.5. reference: https://nvd.nist.gov/vuln/detail/cve-2023-24998

YuMuuu

xuwei-k

**Describe the bug** com.twitter.util.jackson.caseclass.CaseClassDeserializer fails to properly deserialize the following case class. `case class TestRequest(@QueryParam names: Option[Seq[String]])` **To Reproduce** Steps to reproduce the behavior: ``` import com.twitter.finatra.http.HttpServer import com.twitter.finatra.http.routing.HttpRouter object...

pipi-olo

ability to support different request/response types for a route based on the api version

1

**Is your feature request related to a problem? Please describe.** I have a set of endpoints implemented using Finatra, and I'd like to add support for different Request/Response types depending...

alilewin

Fix typo

1

abdullah-afzal