Karol Wójcik
Karol Wójcik
@weavejester @quatrix @chrysophylax Would you please provide a minimal repro to this? I'm happy to do a PR with the fix :)
For completeness, I'm attaching the full stack of what happens: 1. Server hits the swagger-ui ring handler https://github.com/metosin/reitit/blob/master/modules/reitit-swagger-ui/src/reitit/swagger_ui.cljc#L40 2. Reitit uses `response/resource-response` under the hood https://github.com/metosin/reitit/blob/master/modules/reitit-ring/src/reitit/ring.cljc#L237 3. https://github.com/ring-clojure/ring/blob/bc87e3b68b65f5cc2cb3a622239912d0fa575002/ring-core/src/ring/util/response.clj#L337 4. https://github.com/ring-clojure/ring/blob/bc87e3b68b65f5cc2cb3a622239912d0fa575002/ring-core/src/ring/util/response.clj#L355...
Thank you so much @weavejester. I have updated the patch. Hope the changes match `Ring` standards :) BTW: Thank you for making Ring!
@weavejester Is there something I can do more to have it merged?
Don't sorry, it's all good! I'm patient :) Working demo: [here](https://9g7a59k254.execute-api.eu-central-1.amazonaws.com/api-docs/index.html) Source code: [here](https://github.com/FieryCod/holy-lambda-ring-adapter/tree/0.1.0/examples/native) I have included the patch in my library (see [here](https://github.com/FieryCod/holy-lambda-ring-adapter/blob/master/src/fierycod/holy_lambda_ring_adapter/impl.cljc#L22)). **How to test if patch is...
Hi @weavejester! Sorry for not responding for so long. Here is the [repro/example](https://github.com/FieryCod/ring-graalvm-resource-447). Let me know if it's minimal enough. I've also found this issue: https://github.com/ring-clojure/ring/issues/370, which I was not...
This PR is not yet ready to merge. @borkdude found a corner case. When serving a resource directory e.g. `public` at `/public` and trying to access the `/public` the `resource-data`...
Is there any progress in making tide work correctly with tramp?
@jfromaniello ?
@agorgl @skynet-gh I forgot to create an issue on the GraalVM repo. I will try to find some time this week to address it, but I don't promise anything :joy: