Skip to content

Core spec: PullMessagesResponse/CurrentTime description contradicts sibling operations #826

Description

@amakrushin

The description of CurrentTime in PullMessagesResponse is wrong -- The date and time when the messages have been delivered by the web -- devices cannot know the time in future, when their responses will be transferred over the network; also, the description doesn't match the similar descriptions for the CurrentTime in CreatePullPointSubscriptionResponse and RenewResponse:

  • CreatePullPointSubscriptionResponse - Current time of the server for synchronization purposes
  • RenewResponse - The current server time.

Most accurate and useful description in the CreatePullPointSubscriptionResponse.

Also note WS-ResourceLifetime description, more useful and accurate:

The CurrentTime resource property may be used by a client to determine the clock skew between the client and the service, within a margin of error determined by the round-trip latency of the message exchange to retrieve that value. This clock skew and margin of error can then be factored into subsequent decisions of when to send subsequent requests to change the termination time, and what termination times to request.

Suggested fix

Replace the description in PullMessagesResponse and RenewResponse with the description in thee CreatePullPointSubscriptionResponse.

Locations

  • doc/Core.xml 9.1.2 Pull messages, response CurrentTime
  • doc/Core.xml 9.1.3 Renew, response CurrentTime
  • wsdl/ver10/events/wsdl/event.wsdl:159
  • wsdl/ver10/events/wsdl/event-vs.wsdl:159

Related

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions