Skip to content

Commit

Permalink
disable api handling by flask
Browse files Browse the repository at this point in the history
  • Loading branch information
christophlingg committed Oct 31, 2014
1 parent 9896ee1 commit 3269a30
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion website/photon/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ def query_index(query, lang, lon, lat, match_all=True, limit=15):
return es.search(index="photon", body=body)


@app.route('/api/')
#@app.route('/api/')
def api():
lang = request.args.get('lang')
if lang is None or lang not in SUPPORTED_LANGUAGES:
Expand Down

4 comments on commit 3269a30

@yohanboniface
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe let it active when in DEBUG?

@christophlingg
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea, the point is nobody else than us is using this endpoint, we can also rename it as we want. The point is that the public api should be served by the java implementation, because of its new features that the python counterpart does not have.

@yohanboniface
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, but it's still precious (for me at least) for quick prototyping :)

Renaming it sounds good.

@christophlingg
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, we don't like to recompile for development iterations ;-)

Please sign in to comment.