Skip to content

Conversation

@dandical
Copy link

@dandical dandical commented Sep 8, 2025

Adds support for Microsoft Fabric notebook authentication by implementing a new fabricnotebook authentication method.

Similar to the synapsespark method but now uses notebookutils directly to provide support for faster python notebooks.

Changes

  • Added FABRIC_NOTEBOOK_CREDENTIAL_SCOPE constant for Fabric notebook authentication
  • Updated SYNAPSE_SPARK_CREDENTIAL_SCOPE to use "DW" scope instead of the full database URL
  • Implemented get_fabric_notebook_access_token() function that works with both Fabric PySpark and Python notebooks
  • Added fabricnotebook authentication method to AZURE_AUTH_FUNCTIONS mapping

@dandical
Copy link
Author

dandical commented Sep 8, 2025

@dandical please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

@dandical
Copy link
Author

@prdpsvs can we allow all fabric notebooks to support credential pass through like the synapsespark method?

@dandical
Copy link
Author

ill create a new PR with the change from a branch instead of fork

@dandical dandical closed this Sep 12, 2025
@dandical dandical reopened this Sep 12, 2025
@dandical
Copy link
Author

realised need push permissions which so reopened

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants