Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows execution policy not mentioned in docs #44598

Open
bitshiftnetau opened this issue Jan 2, 2022 · 7 comments
Open

Windows execution policy not mentioned in docs #44598

bitshiftnetau opened this issue Jan 2, 2022 · 7 comments

Comments

@bitshiftnetau
Copy link

@bitshiftnetau bitshiftnetau commented Jan 2, 2022

Description

The issue is described here

You cannot possibly be held responsible for all contingencies for every system, that would be unreasonable. However, it would benefit the community, especially those of newcomers, were this issue at least mentioned as known in your docs. A timestamped solution would be most advantageous.

Whether or not this is normal behaviour is irrelevant, most other projects which encounter these issues make mention of the fact. The reality is that this particular problem is not documented and has fallen between the cracks, rendering your getting started doc unhelpful as completing it is left up to chance for the uninitiated. It is cost-free to add this and would speed up the education of those learning the framework.

It serves no-one and nothing to omit this problem and solution from your docs and it should be noted there are 5 or 6 disparate solutions listed on SO, where only one is necessary.

More importantly, these solutions vary in their awareness and consideration of security measures. This alone makes it necessary to guide the user based on your framework's limited requirements.

What is the affected URL?

No response

Please provide the steps to reproduce the issue

On a fresh Windows 10 install, your getting started examples do not work do to limitations on the Execution Policy.

Please provide the expected behavior vs the actual behavior you encountered

Your starting examples should provide adequate information for execution, including minimal permissions required on a system-by-system basis.

Please provide a screenshot if possible

No response

Please provide the exception or error you saw

No response

Is this a browser-specific issue? If so, please specify the device, browser, and version.

No response

@bdshanto
Copy link

@bdshanto bdshanto commented Jan 3, 2022

Need to Execute this
Set-ExecutionPolicy -Scope CurrentUser -ExecutionPolicy RemoteSigned then everything will work fine.
Ref: Set-ExecutionPolicy (Microsoft)

@bitshiftnetau
Copy link
Author

@bitshiftnetau bitshiftnetau commented Jan 3, 2022

Need to Execute this Set-ExecutionPolicy -Scope CurrentUser -ExecutionPolicy RemoteSigned then everything will work fine. Ref: Set-ExecutionPolicy (Microsoft)

Yes. Please add that to the getting started docs for the benefit of new users.

If you wish to absolve yourself of any legal implications, merely add the link to the Execution Policy page, with an example on your docs, and a disclaimer that the developer should seek the advice of a security professional or understand the risks before implementing.

A reminder that omitting this information from the docs only leaves the developer to make often misinformed security decisions. Whilst you cannot be held responsible for that outcome, to ignore any obligation would be equally irresponsible.

@gkalpak
Copy link
Member

@gkalpak gkalpak commented Jan 3, 2022

I think this sounds reasonable, if it's something that people with a fresh/default install will face. Would any of you be interested in creating a pull request?

@ngbot ngbot bot added this to the needsTriage milestone Jan 3, 2022
@ngbot ngbot bot added this to the needsTriage milestone Jan 3, 2022
@yousefalaqra
Copy link

@yousefalaqra yousefalaqra commented Jan 3, 2022

@gkalpak Call for a duty! I'm on it!

@aikithoughts
Copy link
Contributor

@aikithoughts aikithoughts commented Jan 3, 2022

Hi @bitshiftnetau! Thanks for re-filing this issue. (And thanks, @gkalpak for providing your technical insight!)

@yousefalaqra, please feel free to loop me in when your pull request is ready!

yousefalaqra pushed a commit to yousefalaqra/angular that referenced this issue Jan 3, 2022
@yousefalaqra
Copy link

@yousefalaqra yousefalaqra commented Jan 3, 2022

@aikidave
Could you please review the PR

yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 3, 2022
yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 3, 2022
@JoostK JoostK linked a pull request that will close this issue Jan 4, 2022
1 task
yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 4, 2022
yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 4, 2022
yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 4, 2022
yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 4, 2022
yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 4, 2022
yousefalaqra added a commit to yousefalaqra/angular that referenced this issue Jan 4, 2022
Cleans up some of the temporary workarounds that were necessary in order to land support for TypeScript 4.5 since they're no longer necessary.

PR Close angular#44477
@ngtanthanh

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
7 participants
@gkalpak @bitshiftnetau @bdshanto @aikithoughts @yousefalaqra @ngtanthanh and others