Opened 12 years ago
Closed 12 years ago
#507 closed task (fixed)
Possibly get rid of WebOb, Routes, Beaker?
Reported by: | Christopher Allan Webber | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 0.3.3 |
Component: | programming | Keywords: | |
Cc: | Parent Tickets: |
Description
Joar has done some things to switch our requests over to werkzeug's request/response objects. It's possible we should finish the job. For part 1 of this ticket:
- Replace all request and response objects with werkzeug requests/response objects
- Remove webob from setup.py
- Update documentation to reflect
If we do this, then maybe we should just get rid of Beaker and Routes.
Possibly,
- Replace Routes with werkzeug's routing
- Replace Beaker's caching with http://werkzeug.pocoo.org/docs/contrib/cache/
- Replace sessions with itsdangerous http://flask.pocoo.org/snippets/51/
Change History (5)
comment:1 by , 12 years ago
comment:2 by , 12 years ago
Milestone: | → 0.3.3 |
---|---|
Type: | defect → task |
comment:3 by , 12 years ago
OK, reorganized my branches. This is now 2 branches ready for merging, ignore above branch:
Branch: formerge/507_remove_webob
Branch: formerge/507_remove_routes
Repo git://gitorious.org/~spaetz/mediagoblin/spaetz-mediagoblin.git
Routes *are* actually already removed from the code base, but they were still mentioned in the docs. The latter branch removes that.
comment:4 by , 12 years ago
formerge/507_remove_routes' has been merged by Elrond, so the routes packages is not mentioned anymore.
The remove webob branch is left for 0.3.3. After that has been done, we only need to remove beaker, replacing it with werkzeug's sessions.
comment:5 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Merged the "get rid of webob" branch (master changed from 30bb410 to 8d19cb2). The single remaining task is to evaluate the removal of beaker. Filed spinoff issue #580 for this.
I have a branch that completely removes webob ready at 507_remove_webob at repo git://gitorious.org/~spaetz/mediagoblin/spaetz-mediagoblin.git