- 
                Notifications
    You must be signed in to change notification settings 
- Fork 17
[KZL-996] Save Kuzzle API routes in a json file #370
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.
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
Conversation
| Codecov Report
 @@            Coverage Diff            @@
##            6-dev    #370      +/-   ##
=========================================
+ Coverage   96.65%   96.7%   +0.05%     
=========================================
  Files          29      29              
  Lines        1434    1427       -7     
=========================================
- Hits         1386    1380       -6     
+ Misses         48      47       -1
 Continue to review full report at Codecov. 
 | 
| return Promise.resolve(); | ||
| } | ||
|  | ||
| return this._sendHttpRequest('GET', '/') | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not used anymore, you should remove this call because it may throw an error if the anonymous user does not have the server:info right
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Anonymous can use the HTTP protocol since this function will not reject in case of permissions issue. You'll still get a response from the server (with 403 status code yes but a response). This will rejects only in case of network or JSON parsing issue. 🙂
| @scottinet : I put this PR on WIP. After quick talk with @Aschen, it sounds cool to have a new route in Kuzzle server controller to return plugins customs routes. | 
| The issue resolved by this PR here is that, without rights to  Moreover, between  | 
| @Aschen : Does the @scottinet's answer satisfied you? | 
639e6a0    to
    ece089b      
    Compare
  
    To permit the HTTP protocol to not have to performs request on Kuzzle to fetch all available `vanilla` routes.
What does this PR do?
Save Kuzzle API routes in a json file to permit the HTTP protocol to not have to performs request on Kuzzle to fetch all available
vanillaroutes.How should this be manually tested?
Run Unit test and/or use HTTP protocol