search

jakartaee / rest

Jakarta RESTful Web Services
Other
351 stars 114 forks source link

Added List<String> UriInfo#getMatchedResourceTemplates() #1245

Closed jansupol closed 3 months ago

jansupol commented 3 months ago

Alternative to #1236 with a List return type

jansupol commented 3 months ago

@jamezp Do you prefer #1236 over this one?

jamezp commented 3 months ago

@jamezp Do you prefer #1236 over this one?

I do prefer #1236 mainly because it seems to be what the OTel users are looking for. I'll admit it's easier for RESTEasy too, but we could make it work with a collection as well if we needed.

jansupol commented 3 months ago

To me, this one looks more universal, but perhaps it is unnecessary. We can go with the simpler one.

jim-krueger commented 3 months ago

FWIW: This was the response from issue originator (Andrew) on this:

... For our purposes, a String is good, if it was a list we'd want to concatenate it back into a single string.

jansupol commented 3 months ago

Closing this in favor of #1236