dc4d8de77e
Merge branch 'izzy/gql-subscriptions' of https://github.com/Speckle-Next/Server into izzy/gql-subscriptions
2020-08-20 15:26:30 +01:00
22248bf538
test(subs): basic tests for all subscriptions
...
did somebody say TESTS 🥳
2020-08-20 15:26:17 +01:00
80f4dad51e
test(server startup): made it easier? hopefully?
2020-08-20 16:01:19 +03:00
9b02354a42
Merge pull request #18 from Speckle-Next/dim/circle-ci-fixes
...
Dim/circle ci fixes
2020-08-20 15:51:37 +03:00
9ce41e18ff
fix(tests): someone fumbled up
2020-08-20 15:46:17 +03:00
8278f77fbc
test(subs): added missing nodemon dev dep
2020-08-20 15:43:04 +03:00
d40258cfe6
test(circleci): added test server runner separately
2020-08-20 15:39:02 +03:00
ac58d4f53a
Revert "test(subs): bump to timeout 7k"
...
Didn't work.
This reverts commit 41153f1aeb
2020-08-20 10:42:48 +01:00
41153f1aeb
test(subs): bump to timeout 7k to pacify circle ci
2020-08-20 10:36:05 +01:00
762f369c56
test(subs): bump timeout to 5k bc steve jobs's 👻
2020-08-20 10:08:43 +01:00
99f473873a
fix(subs): remove redundant stream sub defs
2020-08-20 10:00:26 +01:00
c6db01420a
test(subs): merge branch 'dim/gql-subscriptions'
2020-08-20 09:50:15 +01:00
7a03c884ea
fix(tests/subscriptions): fixes port conflict in subscription tests when ran from all tests command
2020-08-20 11:31:25 +03:00
29dfa57d82
test(subscriptions): subscription tests now run independently 🙌
2020-08-20 11:26:51 +03:00
e4f213d4f6
test(subscriptions): works with separate run test server, figuring out starting it from test file
...
doing `npm run dev:server:test` and running the test with that server over port 3000 works. seems
like ws upgrade requests are not honoured when the server is started from the test file (on 3001).
2020-08-20 10:27:48 +03:00
45d73e3a47
test(subscriptions): first working scaffold of subscriptions test!
2020-08-19 21:47:48 +03:00
c406abe5ec
feat(subs): subs for branch & commit mutations 🥳
...
also applied @hasRole and @hasScope directives to mutations in this schema
2020-08-19 15:46:40 +01:00
6c3e8b138d
feat(subs): rename stream subs & apply directives
...
- `streamCreated` -> `userStreamCreated
- `streamDeleted` -> `userStreamDeleted`
this is to better show that this is user bound (takes owner id)
rather than stream bound
- @hasRole and @hasScope have been reapplied to each field
2020-08-19 15:45:21 +01:00
222f932547
feat(auth-dirs): revert @hasScope and @hasRole
...
make these field definition only again so it works for now.
the obj directives problem is more messy than i thought 🙃
2020-08-19 15:41:42 +01:00
eabfa99651
feat(auth-dirs): remove @isAuthorized directives
...
was decided that these were a bit too much.
auth will be reverted to `authorizeResolver` within the resolvers
2020-08-19 15:39:52 +01:00
b3cf8e863a
refactor(test/subscriptions): split out the subs test file & added npm test script for it
2020-08-19 17:27:17 +03:00
1f25b8c5fe
refactor(subscriptions): fiddles and riddles
2020-08-19 16:49:39 +03:00
6f788565b7
Merge branch 'izzy/gql-subscriptions' of https://github.com/Speckle-Next/Server into izzy/gql-subscriptions
2020-08-18 18:01:27 +03:00
7c13ab02f2
feat(auth-dirs): do a check for context.auth
2020-08-18 16:00:49 +01:00
1868641367
fix(tests & frontend): fixes the frontend client subscription connection & scaffolds tests
2020-08-18 17:58:07 +03:00
78657bbe7d
feat(subs): finish up streams subscriptions
2020-08-18 10:31:45 +01:00
9bb7b6fe05
feat(auth-dirs): allow hasScope and hasRole on obj
...
implement `visitObject()` so the directives can be applied to objects.
I have applied them to all mutations and tests all still pass.
NOTE: i feel like there must be a way to abstract and reuse directive
definitions as it's very repetitive, but I'm a bit too brain dead today
to think of an elegant solution
2020-08-17 17:12:52 +01:00
3463b998e5
fix(auth-dir): fix silly typo
2020-08-14 16:38:28 +01:00
a4abbe0eaf
test(subs): breaking your tests dim sorry 🙃
...
added three subscription tests that fail b/c i'm dumb and don't know
how the mystical websocket woo woo witchcraft works
2020-08-14 15:36:27 +01:00
ee532b2b96
refactor(streams): use directives for auth
...
comment out any auth logic within the streams resolvers and add auth
directives to the graphql schema instead
test:server all pass for me with this, so hoping everything is ok here!!
2020-08-14 15:32:06 +01:00
6b8bb65dc5
feat(auth-dir): add auth directives to schema
2020-08-14 15:24:47 +01:00
405964b653
feat(auth-dir): break auth into three directives
...
@isAuthorized directive for Stream, Branch, and Commit
this is a light wrapper around `authorizeResolver` which finds the
`resourceId` depending on where the request is coming from.
it's still not super cleand and I will continue thinking about how to
improve this despite inconsistencies with the id location
2020-08-14 15:23:41 +01:00
11c357a79f
fix(directives): find correct resolver
...
handle case when default field resolver is not the right one to use
2020-08-14 10:39:45 +01:00
ec09927f94
feat(directives): add @hasRole directive
2020-08-12 16:03:13 +01:00
392c95d3e1
fix: remove debug: true
...
left this in accidentally as subscriptions are not running as debug for
some reason
2020-08-12 16:02:31 +01:00
e3f2d73444
feat(subs): add stream updated and deleted subs
2020-08-12 16:01:19 +01:00
8838cfd0e4
feat(subs): throw ForbiddenError if no token
2020-08-12 10:57:14 +01:00
0f1dc6078e
fix(directives): typo in hasScope
2020-08-12 10:36:17 +01:00
768ff42a50
refactor(directives): load these nicely
2020-08-12 09:59:07 +01:00
4698612575
feat(subs): copy over regular auth logic for subs
2020-08-11 18:18:35 +01:00
cf6c0e4236
feat(subs): pass header token to connection
2020-08-11 18:17:53 +01:00
9d34ac0e95
feat(subs): return stream input from streamCreated
2020-08-11 17:47:47 +01:00
fa209157bc
feat(subs): add @hasScope dir to streamCreated
2020-08-11 14:39:48 +01:00
ee06bdf63d
feat(directives): add @hasScope to schema
2020-08-11 14:39:06 +01:00
199e576b9f
feat(directives): create HasScopeDirective
2020-08-11 14:36:53 +01:00
fee7ffb257
feat(sub): filter stream created sub by owner id
2020-08-07 15:02:57 +01:00
7b32d39cac
feat(subs): added redis pubsubber (shared resource), imported it into stream resolvers
2020-08-06 10:42:36 +01:00
8f25a6f2fb
fix(subs): subscriptions now properly initialising
2020-08-06 10:16:45 +01:00
d7b75f9bfb
feat(subscriptions): trying to figure out subs
...
I added a streamCreated sub to try it out,
but I can't get it to connect 😅
2020-08-05 17:17:18 +01:00
1fd132c859
style(eslint): added space-unary-ops rule
2020-08-05 16:35:56 +01:00
izzy lyseggen