Skip to content

Conversation

@HKS-MS
Copy link

@HKS-MS HKS-MS commented Jun 11, 2025

I am a Partner Technical Consultant at Microsoft. Today, I worked with a partner who had been facing challenges accessing the private SDK for two weeks. We discovered by chance that the access token had expired, and we struggled to find a way to create a new one so this pull request updates the GET_FACE_ARTIFACTS_ACCESS.md documentation to improve clarity and provide additional usage instructions for generating a new access token and indicates that the access token remains valid for only one week from the creation date.

Documentation improvements:

  • Added an alternative command to generate a new access token using the get_face_access_token.sh script by specifying a #tokenId# parameter.
  • Added an alternative command to generate a new access token using the get_face_access_token.ps1 script with the --tokenId parameter.
  • Included a note clarifying that the access token remains valid for one week from the date of its creation.## Purpose
  • ...

Does this introduce a breaking change?

[ ] Yes
[ x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ x] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@dipidoo
Copy link
Contributor

dipidoo commented Jun 11, 2025

Thank you for your contribution. We have just merged #143 which would address this issue. Do you find that commit satisfactorily address the issue you are targeting with this PR?

As for the expiry, this policy is subject to change driven by another internal process and to avoid staleness of information we have omitted it from information. In any case the expiry will continue to be part of the returned output.

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