Repository navigation
[pull] main from DeusData:main - #159
Merged
Merged
Conversation
Signed-off-by: astandrik <astandrik@yandex-team.ru>
Signed-off-by: astandrik <astandrik@yandex-team.ru>
Signed-off-by: astandrik <astandrik@yandex-team.ru>
Signed-off-by: astandrik <astandrik@yandex-team.ru>
JAX-RS allows @path values without a leading slash, both on the class and on the method. The route extractor rejected such literals, so resources such as `@Path("widgets")` + `@Path("{id}")` produced no Route nodes and no HANDLES edges at all. Accept relative templates and normalize the class/method composition so the resulting path is always rooted. Existing JAX-RS Java probe tests are updated to use relative templates so they guard the regression. Signed-off-by: Hubert Tarnacki <hubert.tarnacki@gmail.com>
Scala JAX-RS resources declare @path on the class exactly like Java and Kotlin, but the class-level prefix was only joined with the method path for those two languages. Include Scala so a method route is emitted as `<class path>/<method path>` instead of the bare method template. Adds a Scala probe test covering both an empty and a non-empty method template. Signed-off-by: Hubert Tarnacki <hubert.tarnacki@gmail.com>
No content change. The previous run of this head failed one test in the macos-14 TSan lane, daemon_runtime_connection_cap_covers_slow_hello_and_ stopping_is_terminal (tests/test_daemon_runtime.c:2846), which this PR does not touch: the same tree passed that lane on 2026-09-13, and a sibling PR on the same base passed it today. The build-config gate this PR adds ran clean in that job. Signed-off-by: Anton Standrik <astandrik@yandex-team.ru>
…path Accepting relative JAX-RS templates made route_path_from_string_node return "" for @path(""), which is non-NULL and so replaced the "/" fallback with an empty method path. @path("") is legal JAX-RS and means "the class path itself"; on main the empty value was rejected and the method fell back to "/" like a method without @path. Return NULL for the empty case so that behaviour is unchanged. The regression covers both shapes: with a class-level @path the method composes to the class path, and without one the route stays "/" (with the empty string it was dropped entirely, because join_route_paths returns the NULL prefix for an empty path). Signed-off-by: Hubert Tarnacki <hubert.tarnacki@gmail.com>
…d-config test: verify sanitizer build configuration
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )