Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

Add --follow support to DELETE command #10

Open
nebhale opened this issue Nov 6, 2012 · 5 comments
Open

Add --follow support to DELETE command #10

nebhale opened this issue Nov 6, 2012 · 5 comments

Comments

@nebhale
Copy link
Contributor

nebhale commented Nov 6, 2012

DELETE can return a 202 with a Location header. The --follow option should exist on the DELETE command to handle this.

jbrisbin pushed a commit that referenced this issue Nov 6, 2012
…king for Location on all HTTP methods but the command itself was hard-coding the follow switch. Added an option to the `delete` and `put` commands to specify the follow switch.
@jbrisbin
Copy link
Contributor

jbrisbin commented Nov 6, 2012

Should be fixed in 3dedc31. Please try and let me know if it doesn't work as expected.

@nebhale
Copy link
Contributor Author

nebhale commented Nov 6, 2012

Doesn't appear to compile at the moment as it's pointing to a snapshot of spring-shell that isn't available.

@jbrisbin
Copy link
Contributor

jbrisbin commented Nov 6, 2012

The binaries use my forked version of spring-shell. If you install that to your local maven repo, it should compile then.

@markpollack
Copy link

I'll review that pull request and get it I to the snapshot build ASAP.

Sent from my iPhone

On Nov 6, 2012, at 11:52 AM, Jon Brisbin [email protected] wrote:

The binaries use my forked version of spring-shell. If you install that to your local maven repo, it should compile then.


Reply to this email directly or view it on GitHub.

@jbrisbin
Copy link
Contributor

jbrisbin commented Nov 6, 2012

I haven't submitted a PR yet but I can. I upgraded SLF4J and Junit too but that stuff can be taken out and reverted to older versions. The important bits are the changes to use java config and smartlifecycle.

Sent from my iPhone

On Nov 6, 2012, at 11:09 AM, Mark Pollack [email protected] wrote:

I'll review that pull request and get it I to the snapshot build ASAP.

Sent from my iPhone

On Nov 6, 2012, at 11:52 AM, Jon Brisbin [email protected] wrote:

The binaries use my forked version of spring-shell. If you install that to your local maven repo, it should compile then.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

3 participants