Skip to content

Fixes #197#198

Merged
defnull merged 2 commits into
bottlepy:masterfrom
sc68cal:issue_197
Aug 26, 2011
Merged

Fixes #197#198
defnull merged 2 commits into
bottlepy:masterfrom
sc68cal:issue_197

Conversation

@sc68cal

@sc68cal sc68cal commented Aug 26, 2011

Copy link
Copy Markdown
Contributor

No description provided.

@sc68cal

sc68cal commented Aug 26, 2011

Copy link
Copy Markdown
Contributor Author

To test:

import bottle

@bottle.get("/foo")
def foo():
    return "bar"

@bottle.get("/bar")
def bar():
    bottle.redirect("/foo")

bottle.debug(True)
bottle.run()
$ telnet localhost 8080
Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
GET /bar HTTP/1.1

HTTP/1.0 303 See Other
Date: Fri, 26 Aug 2011 19:42:16 GMT
Server: WSGIServer/0.1 Python/2.6.1
Content-Type: text/html; charset=UTF-8
Location: http://localhost:8080/foo
Content-Length: 0

Connection closed by foreign host.
$ telnet localhost 8080
Trying 127.0.0.1...
Connected to localhost.
Escape character is '^]'.
GET /bar HTTP/1.0

HTTP/1.0 302 Found
Date: Fri, 26 Aug 2011 19:33:58 GMT
Server: WSGIServer/0.1 Python/2.6.1
Content-Type: text/html; charset=UTF-8
Location: http://localhost:8080/foo
Content-Length: 0

Connection closed by foreign host.

@defnull

defnull commented Aug 26, 2011

Copy link
Copy Markdown
Member

I love clicking that nice green 'merge' button :)

defnull added a commit that referenced this pull request Aug 26, 2011
@defnull defnull merged commit 1af7596 into bottlepy:master Aug 26, 2011
@sc68cal

sc68cal commented Aug 27, 2011

Copy link
Copy Markdown
Contributor Author

Yep, looks like your way is the best. Thanks for your patience. Got that patch in a branch, now let me just add a test case for it.

@defnull

defnull commented Aug 29, 2011

Copy link
Copy Markdown
Member

I'll clean up and simplify a bit, but your pull requests are OK. Merged.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants