docs: 📝 🚀✨ Update README: Added token expiration ⏳ and security tips 🔒! Plus, fresh new images 🌟📸 for token setup—making things clearer and prettier! 🛠️💼 #134
Conversation
…🔒! Plus, fresh new images 🌟📸 for token setup—making things clearer and prettier! 🛠️💼 Closes microsoft#129
|
👋 Thanks for contributing @indcoder! We will review the pull request and get back to you soon. |
There was a problem hiding this comment.
Pull Request Overview
This PR updates the README documentation to provide clearer instructions on creating a token with expiration and adds new security tips along with updated images.
- Introduces an expiration policy for tokens with visual aids.
- Updates headers to reflect keyless authentication and environment variable access.
00-course-setup/README.md
Outdated
| Instead of a token with no expiration date, we recommend creating one for limited days eg: 60 days [if done at the start of April 2025 conveniently coincides with the duration of [AI Skills fest](https://techcommunity.microsoft.com/blog/microsoftlearnblog/register-now-for-the-microsoft-ai-skills-fest/4292261?wt.mc_id=DT-MVP-500304 | ||
|
|
There was a problem hiding this comment.
There appears to be a markdown formatting issue with the link: an extra '[' is leading to an unclosed link syntax. Consider correcting the bracket usage to ensure the link renders properly.
| Instead of a token with no expiration date, we recommend creating one for limited days eg: 60 days [if done at the start of April 2025 conveniently coincides with the duration of [AI Skills fest](https://techcommunity.microsoft.com/blog/microsoftlearnblog/register-now-for-the-microsoft-ai-skills-fest/4292261?wt.mc_id=DT-MVP-500304 | |
| Instead of a token with no expiration date, we recommend creating one for limited days eg: 60 days if done at the start of April 2025 conveniently coincides with the duration of [AI Skills fest](https://techcommunity.microsoft.com/blog/microsoftlearnblog/register-now-for-the-microsoft-ai-skills-fest/4292261?wt.mc_id=DT-MVP-500304). |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
There was a problem hiding this comment.
Please remove your MVP Tracker.
00-course-setup/README.md
Outdated
| Also, as an additional security measure, we can restrict the token to only be used for the `ai-agents-for-beginners` repository [[Least Privilege Principle](https://en.wikipedia. | ||
| org/wiki/Principle_of_least_privilege)]. | ||
|
|
There was a problem hiding this comment.
The markdown link for 'Least Privilege Principle' is split across two lines, which may break the link rendering. Moving the entire URL to a single line should resolve this issue.
| Also, as an additional security measure, we can restrict the token to only be used for the `ai-agents-for-beginners` repository [[Least Privilege Principle](https://en.wikipedia. | |
| org/wiki/Principle_of_least_privilege)]. | |
| Also, as an additional security measure, we can restrict the token to only be used for the `ai-agents-for-beginners` repository [[Least Privilege Principle](https://en.wikipedia.org/wiki/Principle_of_least_privilege)]. |
Copilot is powered by AI, so mistakes are possible. Review output carefully before use.
…🔒! Plus, fresh new images 🌟📸 for token setup Closes microsoft#129
…rs into fga_token
…rs into fga_token
…rs into fga_token
|
I would request to retain it.. Infact thanks to your comment I noticed there was an error in it because I rushed to put it in. I added that so that Microsoft can track these micro nudges from MS Learn Beginner Series to Program Initiatives like AI Skills Fest, Season of Agents etc and I have reminder for a follow up PR on 31st May 2025 to remove it. This PR is important to inculcate Zero Trust and LPA with college students for which we are beginning tomorrow https://x.com/augvcor/status/1909194849183543461 |
|  | ||
|
|
||
| Also, as an additional security measure, we can restrict the token to only be used for the `ai-agents-for-beginners` repository [[Least Privilege Principle](https://en.wikipedia.org/wiki/Principle_of_least_privilege)]. | ||
|  |
There was a problem hiding this comment.
nit: can we include more visually appealing images instead of annotations draw by hand/mouse? the "Generate Token" image looks good.
There was a problem hiding this comment.
Agree here - would be helpful with some quick improvements
Closes #129