Update odsp driver for new header

[MarioJGMsoft]

Description

A new header variable is necessary when redeeming a share link so that a certain type of user has temporal access to the document to which the link points to.

Reviewer Guidance

Let me know if there's something I need to add or be aware of.

Fixes: AB#41357

[Copilot]

Pull Request Overview

Adds support for a new isNonDurableRedeem header to enable temporal, non-durable access via share links. Key changes include:

• Extending the resolver and contracts to extract and propagate the new header.
• Modifying snapshot redemption and download flows to honor the non-durable flag.
• Updating public API definitions and adding unit tests for the new behavior.

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
packages/drivers/odsp-driver/src/test/odspDriverUrlResolverForShareLink.spec.ts	Added tests verifying isNonDurableRedeem is set or ignored
packages/drivers/odsp-driver/src/odspDriverUrlResolverForShareLink.ts	Extract and attach isNonDurableRedeem to shareLinkInfo
packages/drivers/odsp-driver/src/fetchSnapshot.ts	Propagate isNonDurableRedeem into prefer headers for redeem and download APIs
packages/drivers/odsp-driver/src/contractsPublic.ts	Extended SharingLinkHeader enum and internal interface
packages/drivers/odsp-driver-definitions/src/resolvedUrl.ts	Added isNonDurableRedeem to ShareLinkInfoType
packages/drivers/odsp-driver-definitions/api-report/odsp-driver-definitions.legacy.alpha.api.md	Updated API report with new field

Comments suppressed due to low confidence (4)

packages/drivers/odsp-driver/src/contractsPublic.ts:37

• This header may be omitted in normal requests. Consider making the property optional ([SharingLinkHeader.isNonDurableRedeem]?: boolean) to reflect its conditional presence.

[SharingLinkHeader.isNonDurableRedeem]: boolean;

packages/drivers/odsp-driver/src/fetchSnapshot.ts:740

• [nitpick] Rename header to headers to clarify that this object holds multiple HTTP headers and align with the naming used elsewhere in the file.

const header = isNonDurableRedeem

packages/drivers/odsp-driver/src/test/odspDriverUrlResolverForShareLink.spec.ts:373

• [nitpick] The assertion message is generic. Consider specifying that you expect the isNonDurableRedeem flag to be true (e.g., Expected isNonDurableRedeem to be true).

"Sharing link should be set in resolved url",

packages/drivers/odsp-driver/src/test/odspDriverUrlResolverForShareLink.spec.ts:390

• [nitpick] This assertion message could more clearly reference isNonDurableRedeem (e.g., Expected isNonDurableRedeem to be undefined).

"Sharing link should not be set in resolved url",

[Josmithr]

@defaultValue is intended to describe the behavior that results from not specifying a value (or setting it to undefined). So here it should describe the default boolean value that will be used when this is undefined.

[github-actions]

🔗 No broken links found! ✅

Your attention to detail is admirable.

linkcheck output

> fluid-framework-docs-site@0.0.0 ci:check-links /home/runner/work/FluidFramework/FluidFramework/docs
> start-server-and-test "npm run serve -- --no-open" 3000 check-links

1: starting server using command "npm run serve -- --no-open"
and when url "[ 'http://127.0.0.1:3000' ]" is responding with HTTP status code 200
running tests using command "npm run check-links"


> fluid-framework-docs-site@0.0.0 serve
> docusaurus serve --no-open

[SUCCESS] Serving "build" directory at: http://localhost:3000/

> fluid-framework-docs-site@0.0.0 check-links
> linkcheck http://localhost:3000 --skip-file skipped-urls.txt

Crawling...

Stats:
  225443 links
    1710 destination URLs
    1941 URLs ignored
       0 warnings
       0 errors