这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@nizar-m
Copy link
Contributor

@nizar-m nizar-m commented Oct 12, 2018

Description

What component does this PR affect?

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Requires changes from other components? If yes, please mark the components:

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Related Issue

issue #196

Solution and Design

Adding tests

  • With access key
  • With access key and webhook

All the tests are rerun with the required headers for access control.

A local HTTPS webhook is used (webhook.py in server/tests-py folder). It's certificate authority is copied to /etc/ssl/certs so that the webhook is deemed to be secure.

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Docs update
  • Community content
  • Tests

Checklist:

  • I have read the contributing guide and my code conforms to the guidelines.
  • This change requires a change in the documentation.
  • I have updated the documentation accordingly.
  • I have added required tests.

@CLAassistant
Copy link

CLAassistant commented Oct 12, 2018

CLA assistant check
All committers have signed the CLA.

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@shahidhk shahidhk added the c/server Related to server label Oct 12, 2018
@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@nizar-m nizar-m changed the title Tests for server with access control Tests for server with access control, and some more tests Oct 26, 2018
@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@0x777 0x777 merged commit 0ffb047 into hasura:master Oct 28, 2018
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-710.herokuapp.com is deleted

@hasura-bot
Copy link
Contributor

Review app available at: https://hge-ci-pull-710.herokuapp.com

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 30, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 964dec9
Status:⚡️  Build in progress...

View logs

hasura-bot pushed a commit that referenced this pull request Jun 12, 2024
Rules:
* The schema ID is set by the `id` property, falling back to `rename`.
* The schema ID is always prefixed by a URL base.
* The schema name is set by the `rename` property, falling back to `id`.
* The schema title is set by the `title` property, falling back to
`rename`, then `id`.
* If the name or title are automatically created from the ID, remove
characters that might choke a code generator, such as ' ' or '/'.
V3_GIT_ORIGIN_REV_ID: fac46a0849c2f36642daeca2971fdce7d253256a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/server Related to server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants