Breaking change introduced in the latest draft - `endpoint_url` is renamed to `url`
Closed this issue · 2 comments
appsdesh commented
tulshi commented
I'm trying to understand the impact of this change, since it is only related to the metadata and it is used in the examples in the previous implementer's draft (although the section defined it as "endpoint_url"). Could you please comment on whether this is impactful to existing implementations or hard to update?
appsdesh commented
I'm trying to understand the impact of this change, since it is only related to the metadata and it is used in the examples in the previous implementer's draft (although the section defined it as "endpoint_url"). Could you please comment on whether this is impactful to existing implementations or hard to update?
@atultulshi summarizing our offline discussions -
- This PR will fix the ambiguity between attribute names between metadata and examples from the last implementers draft.
- https://sharedsignals.guide/ is a great guide that also uses
endpoint_url
in configuring the streams, without this PR's fix we are burdening existing transmitters to change the implementation (for no reason whatsoever). This is a breaking change that got introduced in the current draft and we should fix it. - Since
url
overenpoint_url
does not add any functional value but only adds brevity in naming, we should stick to the previous implementer's draft and NOT introduce any breaking changes.