Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove old API #278

Merged
merged 64 commits into from
Mar 30, 2015
Merged

Remove old API #278

merged 64 commits into from
Mar 30, 2015

Conversation

rochoa
Copy link
Contributor

@rochoa rochoa commented Mar 30, 2015

This implies a MAJOR RELEASE with BREAKING CHANGES

  • Removes /:table/infowindow, /:table/map_metadata and /:table/flush_cache endpoints
  • Sample configuration removes /tiles/template and /tiles/layergroup
  • URLs to use from now on are: /api/v1/map/named and /api/v1/map
  • No more state changes for styles
  • No more dump stats for renderers: SIGUSR1 and SIGUSR2 signals
  • Removes query params:
    • sql
    • geom_type
    • cache_buster
    • cache_policy
    • interactivity
    • style
    • style_version
    • style_convert
    • scale_factor
  • Affected tables for x-cache-channel will use direct connection to postgresql
  • Removes some metrics: authorized times ones

This closes #257 #52 #101 #102

@rochoa
Remove infowindow endpoint
ba0f394
@rochoa
Removes map_metadata endpoint
a4eade3
@rochoa
Removes flush_cache endpoint
636591e
@rochoa
Remove geom_type retrieval
59dfd11
@rochoa
Rely on windshaft's remove old api branch
a0243e0
@rochoa
No more before/after state change actions as there is no longer
597cb52 
a style change
@rochoa
Remove dump render cache stats
a2a0997
@rochoa
Merge branch 'master' into 257-remove-old-api
71efe21 
Conflicts:
	lib/cartodb/cartodb_windshaft.js
	package.json
@rochoa
Merge branch 'master' into 257-remove-old-api
ef2b456 
Conflicts:
	package.json
@rochoa
Merge branch 'master' into 257-remove-old-api
5f6185d 
Conflicts:
	lib/cartodb/cartodb_windshaft.js
	lib/cartodb/server_options.js
	package.json
@rochoa
Re-generate npm-shrinkwrap.json to use windshaft/master.
0321884 
A lot of tests from acceptance/server.js will fail
@rochoa
jshint
25e4e3b
@rochoa
Remove get style tests
6d32199
@rochoa
Merge branch 'master' into 257-remove-old-api
f311f6d
@rochoa
PHONY coverage
0e3f72c
@rochoa
Remove per-table varnish invalidation
02bc7b9
@rochoa
Remove cdbQueryTablesFromPostgres option, now uses it by default
07297f6
@rochoa
Remove config and sql api backend
23f5be6
@rochoa
Deprecates old config URLs: /tiles/template and /tiles/layergroup
d9ab1e8 
Moves all tests to run on new URLs
Deprecated base_url_legacy in config, it will keep working tho
@rochoa
Skip server suite for now until we decide what tests we should port
09b3f0a
@rochoa
Move userByReq to its own model
f5f7be6
@rochoa
Remove some old accepted query params
c19f652
@rochoa
jshint
415d0c4
@rochoa
No sql param in generateCacheChannel
ea3d212
@rochoa
No longer possible to set sql param
e1e5f87
@rochoa
no longer possible to set cache_buster request param
36ea58e
@rochoa
Remove cache_policy query param
d5e20ef
@rochoa
Remove table param from generateCacheChannel
1d433bf
@rochoa
Remove check table privacy
07bc281
@rochoa
Split old api and basic endpoints
985973d
@rochoa
no need to check it used post for long queries
848bfac
@rochoa
No need to have a test for empty cartocss
ff515f8
@rochoa
Removes test as is covered in "layergroup creation fails if CartoCSS …
000a248 
…is bogus"
@rochoa
jshint adds mocha globals
ada616e
@rochoa
Replace server test with multilayer one
e7544e8
@rochoa
it instead of test
4ea1199
@rochoa
Replace test for multiple cartocss errors
ecc67b1
@rochoa
No need to test good style cases
ec2fcad
@rochoa
No need to test style deletion
41521b6
@rochoa
Remove comments
2161bbf
@rochoa
style
70d1a30
@rochoa
Removes tests as they are covered in multilayer suite
2b9f2ee
@rochoa
Replace zoom test with multilayer one
124133c
@rochoa
Removes sql api specific test
55a7889
@rochoa
Extract request function
434d6d4
@rochoa
Replaces server tests for user db connection with multilayer
d6ef0b7
@rochoa
Replace lzma test with multilayer
c92e786
@rochoa
Invalid json lzma test
7df3658
@rochoa
Fixes jshint
85804f9
@rochoa
Replaces affected tables test with multilayer one
b599e67
@rochoa
Replace affected tables tests with multilayer ones
1fc0545
@rochoa
Tests for cdb request
f1e8c9a
@rochoa
Mock better query tables api
69f99da
@rochoa
Tests for surrogate keys invalidation
a2ac1c2 
 - uses nock to mock request
 - a bit hacky because tests keep environment inconsistent
   * for intance I had to mock cartocdn to not fail in other suite
@rochoa
Regenerate npm-shrinkwrap.json
cd79fc6
@rochoa
Prepared next release
e0413c3
rochoa pushed a commit that referenced this pull request Mar 30, 2015
Merge pull request #278 from CartoDB/257-remove-old-api
3234c37 
Remove old API
@rochoa rochoa merged commit 3234c37 into master Mar 30, 2015
@rochoa rochoa deleted the 257-remove-old-api branch March 30, 2015 17:04
@rochoa rochoa mentioned this pull request Mar 30, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

remove old api
1 participant
@rochoa