Opened on 03/20/2015 at 08:56:02 PM

Closed on 03/23/2015 at 12:52:44 PM

#2196 closed change (fixed)

[cms] Drop depencency on Flask for test server

Reported by: trev Assignee: trev
Priority: P4 Milestone:
Module: Sitescripts Keywords:
Cc: Blocked By:
Blocking: #2197 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working:
Review URL(s):

http://codereview.adblockplus.org/6451712864813056/

Description

Background

We currently require Flask to run our test server. However, the CMS web app is very simple, the Flask framework is an overkill.

What to change

Use the werkzeug.simple module directly instead of calling it via Flask. If Werkzeug isn't installed, fall back to the wsgiref module that is built into Python - compared to Werkzeug this one merely lacks developer features.

Attachments (0)

Change History (3)

comment:1 Changed on 03/20/2015 at 09:12:31 PM by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed on 03/20/2015 at 10:31:31 PM by trev

  • Blocking 2197 added

comment:3 Changed on 03/23/2015 at 12:52:44 PM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
 
Note: See TracTickets for help on using tickets.