Skip to content
This repository was archived by the owner on Feb 26, 2023. It is now read-only.
This repository was archived by the owner on Feb 26, 2023. It is now read-only.

@RequiresHeader not working with HTTP DELETE #706

Closed
Closed
@RequiresHeader not working with HTTP DELETE#706
Labels
ContributionWelcomeDefect
Milestone
3.0

Description

@codebreaker
codebreaker
opened on Aug 28, 2013

I'm using 3.0-milestone build and it looks like using @RequiresHeader for a HTTP DELETE is not adding headers. I have checked in the generated rest service implementation, the methods generated for DELETE the are not adding headers to the request.

Activity

  1. DayS commented on Sep 1, 2013

    @DayS
    DayS
    on Sep 1, 2013
    Contributor

    Ho. It seems this annotation is only handle for Get, Post and Put http methods. We should fix this.

  2. DayS commented on Sep 1, 2013

    @DayS
    DayS
    on Sep 1, 2013
    Contributor

    This is related to #475

  3. dhleong commented on Sep 3, 2013

    @dhleong
    dhleong
    on Sep 3, 2013
    Contributor

    Oh shoot, I didn't see this issue until just now. I opened a pull request referencing the old #475 ticket: #711

  4. lalith-b commented on Sep 20, 2013

    @lalith-b
    lalith-b
    on Sep 20, 2013
    Contributor

    HTTP DELETE & PUT has no return type. Always VOID. Is this a bug ?

  5. DayS commented on Sep 20, 2013

    @DayS
    DayS
    on Sep 20, 2013
    Contributor

    It might be yes. We should handle response for delete.

  6. DayS commented on Oct 12, 2013

    @DayS
    DayS
    on Oct 12, 2013
    Contributor

    Fixed

  7. DayS
    closed this as completedon Oct 12, 2013
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

    ContributionWelcomeDefect

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions