cubic-js / cubic-api Goto Github PK

Either remove slash from url & route or add one explicitly

Ditch resource array. Give URL directly for more flexibility.
-> this.url = "/some/path/:id/does/something/with/:name/query

This makes resource to url conversion redundant, but resource keys need to be extracted from URL now.

Readme still contains irrelevant warframe-nexus information. Adapt to new style.

The devDependency cubic-core was updated from 1.1.4 to 1.1.5.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

cubic-core is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Your Greenkeeper Bot 🌴

From @Kaptard on April 25, 2017 4:24

From @Kaptard on April 15, 2017 20:43

Implement some sort of cache to reduce input to resource-node for most-used methods.

Copied from original issue: nexus-devs/NexusHub#5

Copied from original issue: cubic-js/cubic#4

use socket.on("endpoint/url", data) instead of passing object on new event

Current way of handling scoping seems clunky considering the use of more specified actions (i.e. managing user profiles). Need to analyze existing solutions and draw fitting conclusion.

The devDependency cubic-loader was updated from 1.2.0 to 1.2.1.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

cubic-loader is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Your Greenkeeper Bot 🌴

like /some/:id/resource. Would amend recent URL-specific middleware issue.

Emit resource location when emitting check. Core node should only ack if local resource available. Otherwise multiple different nodes will get requests they can't respond to.

https://swagger.io

See title. Adds additional document instead.

Routes aren't mapped on config event, only pulling from db.

Not sure why. It does return the correct data. But it's not saved Properly. Possibilities are:

• Getting the wrong url in the endpoint object
• Using a previously saved url on cache update
• Redis incorrectly saving data (unlikely)
• Redis swapping values (very unlikely)

use /foo instead of http://nexus-stats.com/foo

From @Kaptard on April 25, 2017 14:47

e.g.

blitz.get("/register", (req, res, next) => {...})

Copied from original issue: cubic-js/cubic#7

Either doesn't load or doesn't get routed on express

From @Kaptard on May 2, 2017 16:35

Otherwise quickstart is a pain in the ass

Copied from original issue: cubic-js/cubic-auth#2

The devDependency cubic-client was updated from 1.1.6 to 1.1.7.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

cubic-client is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Your Greenkeeper Bot 🌴

See Express. Need to return function instead of calling immediately -> blitz.api.use(middleware())

Title. Just don't write the published message to clients that don't have the required scope.

Would be triggered on same event as Socket.io publish.
Note: This isn't particularily urgent right now since WebSockets provide the same functionality already.

Fragment from nexus-stats. says 'refer to api.nexus-stats.com'. Use blitz.config.core.apiURL instead.

This would allow adjusting options before server-setup. Clients won't connect to custom locations otherwise.

From @Kaptard on April 25, 2017 14:44

Need option to use combined middleware on certain routes only.

Copied from original issue: cubic-js/cubic#6

The devDependency cubic-api was updated from 1.3.1 to 1.3.2.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

cubic-api is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.

Your Greenkeeper Bot 🌴

From @Kaptard on April 25, 2017 4:23

From @Kaptard on April 24, 2017 7:37

Invalid requests return no information about rejection reason.

To fix: Return Error Object with proper reason. If return value instance of error -> send error back.

Copied from original issue: nexus-devs/NexusHub#9

Copied from original issue: cubic-js/cubic#2

🚨 You need to enable Continuous Integration on all branches of this repository. 🚨

To enable Greenkeeper, you need to make sure that a commit status is reported on all branches. This is required by Greenkeeper because it uses your CI build statuses to figure out when to notify you about breaking changes.

Since we didn’t receive a CI status on the greenkeeper/initial branch, it’s possible that you don’t have CI set up yet. We recommend using Travis CI, but Greenkeeper will work with every other CI service as well.

If you have already set up a CI for this repository, you might need to check how it’s configured. Make sure it is set to run on all new branches. If you don’t want it to run on absolutely every branch, you can whitelist branches starting with greenkeeper/.

Once you have installed and configured CI on this repository correctly, you’ll need to re-trigger Greenkeeper’s initial pull request. To do this, please delete the greenkeeper/initial branch in this repository, and then remove and re-add this repository to the Greenkeeper App’s white list on Github. You'll find this list on your repo or organization’s settings page, under Installed GitHub Apps.

endpoints.js currently doesn't convert bool-like strings into actual bool types.

Suggestion:
"true" -> true
"false" -> false
1 -> true
0 -> false

Check back on this after Web-Node is connected to API node or has own API for user-handling.

Core node does successfully connect to root namespace, but won't work until full reconnect (node restart)

Title. Goes along with #7

From @Kaptard on April 25, 2017 4:24

From @Kaptard on April 15, 2017 20:41

Min-diff between requests still working.
Bug occurred after refactoring auth.js into auth & limit.

Copied from original issue: nexus-devs/NexusHub#4

Copied from original issue: cubic-js/cubic#5