Skip to content

Commit

Permalink
fix(config): protected directive on object (#1728)
Browse files Browse the repository at this point in the history
  • Loading branch information
ssddOnTop authored Apr 15, 2024
1 parent ede9a3a commit c587c4a
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
6 changes: 6 additions & 0 deletions src/config/into_document.rs
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,12 @@ fn config_document(config: &ConfigModule) -> ServiceDocument {
.as_ref()
.map(|cache| pos(cache.to_directive())),
)
.chain(
type_def
.protected
.as_ref()
.map(|protected| pos(protected.to_directive())),
)
.chain(type_def.tag.as_ref().map(|tag| pos(tag.to_directive())))
.collect::<Vec<_>>(),
kind,
Expand Down
2 changes: 1 addition & 1 deletion tests/snapshots/execution_spec__auth-basic.md_merged.snap
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ type Nested {
protected: String! @protected
}

type ProtectedType {
type ProtectedType @protected {
name: String!
nested: String!
}
Expand Down
2 changes: 1 addition & 1 deletion tests/snapshots/execution_spec__auth-jwt.md_merged.snap
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ type Nested {
protected: String! @protected
}

type ProtectedType {
type ProtectedType @protected {
name: String!
nested: String!
}
Expand Down

1 comment on commit c587c4a

@github-actions
Copy link

Choose a reason for hiding this comment

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

Running 30s test @ http://localhost:8000/graphql

4 threads and 100 connections

Thread Stats Avg Stdev Max +/- Stdev
Latency 7.27ms 5.68ms 188.19ms 93.65%
Req/Sec 3.58k 149.82 3.89k 93.50%

427202 requests in 30.00s, 2.14GB read

Requests/sec: 14237.78

Transfer/sec: 73.08MB

Please sign in to comment.