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

Update Editor to 2017.1.0p5 for Security Update & VS compatibility #888

Merged
merged 5 commits into from
Aug 25, 2017
Merged

Update Editor to 2017.1.0p5 for Security Update & VS compatibility #888

merged 5 commits into from
Aug 25, 2017

Conversation

StephenHodgson
Copy link
Contributor

@StephenHodgson StephenHodgson commented Aug 24, 2017

Fixes #849
Fixes #860

@StephenHodgson StephenHodgson changed the title Update Editor to 2017.1.0p4 for Security Update Update Editor to 2017.1.0p5 for Security Update & VS compatibility Aug 25, 2017
@@ -607,7 +611,7 @@ PlayerSettings:
EnterpriseAuthentication: False
HumanInterfaceDevice: False
InputInjectionBrokered: False
InternetClient: False
InternetClient: True
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting that this hadn't already been set!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought so too!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ohh interesting, this must be why we've never noticed it (here under Internet and public networks):

The internetClientServer capability includes the access that the internetClient capability provides, so you don't need to specify internetClient when you specify internetClientServer.

Probably doesn't hurt to keep it set though!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh I see.

Yeah doesn't matter either way I suppose. Users will still need to set it in their projects anyway.

Copy link
Contributor

@ForrestTrepte ForrestTrepte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@StephenHodgson StephenHodgson merged commit b77eabc into microsoft:master Aug 25, 2017
@StephenHodgson
Copy link
Contributor Author

I'll make a new release this weekend addressing this update.

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.

None yet

4 participants