azure-functions-core-tools icon indicating copy to clipboard operation
azure-functions-core-tools copied to clipboard

Adding additional command line param of sync-access-token

Open aneillans opened this issue 1 year ago • 1 comments

Issue describing the changes in this PR

Related to the issue linked below, adds the ability to pass a different access token to be used for the sync triggers call.

partially resolves #4196

Pull request checklist

  • [ ] My changes do not require documentation changes
    • [ ] Otherwise: Documentation issue linked to PR
  • [X] My changes do not need to be backported to a previous version
    • [ ] Otherwise: Backport tracked by issue/PR #issue_or_pr
  • [ ] I have added all required tests (Unit tests, E2E tests)

aneillans avatar Nov 21 '24 20:11 aneillans

@soninaren do you mind taking a look at this

liliankasem avatar Mar 12 '25 22:03 liliankasem

Sorry it's taken so long to get to this.

The App Service Authentication layer shouldn't come into play here. The Core Tools calls ARM, which in turn calls an app admin endpoint which is not subject to user-defined auth checks.

liliankasem avatar Nov 03 '25 21:11 liliankasem

Sorry it's taken so long to get to this.

The App Service Authentication layer shouldn't come into play here. The Core Tools calls ARM, which in turn calls an app admin endpoint which is not subject to user-defined auth checks.

If that's the case, then there are some issues with the kudu deployment process (would need to go back and check my notes on the issue I was hitting at the time).

Deployments behind fully authentication enabled do fail to correctly sync due to these calls not having the correct Auth passed.

aneillans avatar Nov 04 '25 04:11 aneillans