-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21460/
-----------------------------------------------------------

(Updated May 14, 2014, 9:42 p.m.)


Review request for Aurora and Kevin Sweeney.


Changes
-------

Because of a feature/bug in our Http Registration framework, we can't 404 on 
/api/ but serve /api. Since users can still query on /api/* and will receive a 
uncompressed response which can cause subtle page load issues that can be hard 
to detect. So, adding gzip encoding to /api/* also.


Repository: aurora


Description
-------

Removed trailing / from /api end point. Fixing a bad merge from 
https://reviews.apache.org/r/18658/

Removing the /, re-enables the gzip encryption.


Diffs (updated)
-----

  src/main/java/org/apache/aurora/scheduler/thrift/ThriftModule.java 
c14d5c2d059e8970b2f232b7a2a0958ba8cffcb8 
  src/main/resources/org/apache/aurora/scheduler/http/ui/js/services.js 
feb9ece3ec324840c33e0449f27e9914480aa061 

Diff: https://reviews.apache.org/r/21460/diff/


Testing
-------

Tested on laptop.


File Attachments (updated)
----------------

gzip encoded /api end point
  
https://reviews.apache.org/media/uploaded/files/2014/05/14/2ea6ea61-0740-491c-a016-596888f56b7c__Screen_Shot_2014-05-14_at_2.26.17_PM.png
Enabled gzip encoding for /api/  end point.
  
https://reviews.apache.org/media/uploaded/files/2014/05/14/637c4f85-7d22-49c1-89fc-7fea74f36b29__Screen_Shot_2014-05-14_at_2.37.49_PM.png


Thanks,

Suman Karumuri

Reply via email to