Skip to content

[WebAuthn] Add tests for cross-origin create()#43729

Merged
stephenmcgruer merged 5 commits intomasterfrom
smcgruer/webauthn-cross-origin-tests
Jan 18, 2024
Merged

[WebAuthn] Add tests for cross-origin create()#43729
stephenmcgruer merged 5 commits intomasterfrom
smcgruer/webauthn-cross-origin-tests

Conversation

@stephenmcgruer
Copy link
Contributor

@stephenmcgruer stephenmcgruer force-pushed the smcgruer/webauthn-cross-origin-tests branch from c5d1e96 to 3763cc2 Compare December 20, 2023 16:32
@stephenmcgruer stephenmcgruer marked this pull request as ready for review December 20, 2023 16:36
@stephenmcgruer
Copy link
Contributor Author

This test should pass in WPT's Chrome Dev setup once it reaches 122.0.6196.0 (which may not be until after the holidays). It passes locally for me using Chrome Canary. I'm happy to wait until Chrome Dev catches up before landing, if desirable :)

Copy link
Member

@nsatragno nsatragno left a comment

Choose a reason for hiding this comment

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

Thanks for the WPT!

@stephenmcgruer
Copy link
Contributor Author

@stephenmcgruer
Copy link
Contributor Author

@kenrb if you can take a look in Nina's absence, that would be great thanks!

Copy link

@kenrb kenrb left a comment

Choose a reason for hiding this comment

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

lg although one change might have been missed?

@stephenmcgruer
Copy link
Contributor Author

Thanks Ken! Assigning to @rsolomakhin for LGTM, since looks like Ken maybe isn't a reviewing member of WPT?

@community-tc-integration
Copy link

Uh oh! Looks like an error! Details

Client ID static/taskcluster/github does not have sufficient scopes and is missing the following scopes:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w/fdLXHGJsRnyOiJ6X4vch1g",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w"
      ]
    }
  ]
}

This request requires the client to satisfy the following scope expression:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w/fdLXHGJsRnyOiJ6X4vch1g",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w"
      ]
    }
  ]
}

  • method: rerunTask
  • errorCode: InsufficientScopes
  • statusCode: 403
  • time: 2024-01-18T15:31:58.319Z

@community-tc-integration
Copy link

Uh oh! Looks like an error! Details

Client ID static/taskcluster/github does not have sufficient scopes and is missing the following scopes:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w/fmI5mb-CSxWdKAjwXt2wCA",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w"
      ]
    }
  ]
}

This request requires the client to satisfy the following scope expression:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w/fmI5mb-CSxWdKAjwXt2wCA",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w"
      ]
    }
  ]
}

  • method: rerunTask
  • errorCode: InsufficientScopes
  • statusCode: 403
  • time: 2024-01-18T15:31:58.618Z

@stephenmcgruer
Copy link
Contributor Author

The failure from Firefox stability is from the mozilla side:

requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: https://hg.mozilla.org/mozilla-central/archive/tip.zip/testing/profiles/

So I'm going to merge this.

@stephenmcgruer stephenmcgruer enabled auto-merge (squash) January 18, 2024 15:32
@community-tc-integration
Copy link

Uh oh! Looks like an error! Details

Client ID static/taskcluster/github does not have sufficient scopes and is missing the following scopes:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w/fdLXHGJsRnyOiJ6X4vch1g",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w"
      ]
    }
  ]
}

This request requires the client to satisfy the following scope expression:

{
  "AnyOf": [
    "queue:rerun-task:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w/fdLXHGJsRnyOiJ6X4vch1g",
    "queue:rerun-task-in-project:none",
    {
      "AllOf": [
        "queue:rerun-task",
        "assume:scheduler-id:taskcluster-github/aw_8Vr8FQrC1VpewZElt_w"
      ]
    }
  ]
}

  • method: rerunTask
  • errorCode: InsufficientScopes
  • statusCode: 403
  • time: 2024-01-18T15:32:58.293Z

auto-merge was automatically disabled January 18, 2024 15:33

Pull request was closed

@stephenmcgruer stephenmcgruer merged commit d46f1ae into master Jan 18, 2024
@stephenmcgruer stephenmcgruer deleted the smcgruer/webauthn-cross-origin-tests branch January 18, 2024 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants