Skip to content

Fix Service invocation path segments automatic encode of / into %2f #702

Description

@mukundansundar

Expected Behavior

Given a method name containing "/" in service invocation, properly split path segments and call dapr.

Actual Behavior

Currently in PubSub IT,
https://github.com/dapr/java-sdk/blob/master/sdk-tests/src/test/java/io/dapr/it/pubsub/http/PubSubIT.java#L396-L400

The method being called is messages/ttlTopic which gets encoded as messages%2fttlTopic which was working prior to fix dapr/dapr#4008.

With the fix merged to Dapr, on Feb 8th, any build of Dapr after that fails the PubSub IT since it tries to invoke http://...../messages%2fttltopic instead of http://..../messages/ttlTopic.

See https://github.com/dapr/java-sdk/runs/5533656018?check_suite_focus=true#step:20:6305

Steps to Reproduce the Problem

Change Dapr Ref in build.yaml to any latest master ref from dapr/dapr and run the ITs.

Release Note

RELEASE NOTE: Fix service invocation path segments to avoid automatic encoding of / to %2f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions