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 page "Configure server-to-server authentication between publishing and consuming farms" #298

Merged
merged 8 commits into from
Oct 4, 2018

Conversation

Yvand
Copy link
Contributor

@Yvand Yvand commented Oct 4, 2018

Current version of the article configures all SharePoint farms to use the same authentication realm, which causes problems:

  • It may cause issues with workloads that rely on OAuth (e.g. add-ins, Workflow Manager, Office Online Server).
  • All farms get full trust permission between each other and can do everything on each other.

I changed the procedure to preserve the original unique authentication realm on every SharePoint farm, and set just the permissions actually required for the social features to work.

Yvand added 8 commits October 2, 2018 16:05
The original article configures all SharePoint farms to use the same authentication realm, which is bad as it may cause issues with over workloads that rely on OAuth (e.g. add-ins, Workflow Manager, Office Online Server).
I reviewed the procedure to not modify the authentication realm if any farm which eliminates issue above, and I just set the permissions actually required for the OAuth requests to work (versus full trust when authentication realm is unique across farms).
@kenwith
Copy link
Member

kenwith commented Oct 4, 2018

@Yvand - thank you for the work on this change and taking the time to make the docs better. It is much appreciated!

@AndreaBarr - can you get this into the backlog for the SharePoint team to review? This is a Pull Request but fairly significant change so the writer that owns this article will need to handle it.

@Yvand
Copy link
Contributor Author

Yvand commented Oct 4, 2018

@kenwith my pleasure, after so much effort I thought I had to share this!
It's indeed a significant change, I worked incrementally to grant only permissions that are actually required, and made quite a lot of tests to ensure that all social features work.

In case you are looking for a not-too-complicated way to repro this, I started from this Azure template, provisioned it with a front-end, removed the front-end from the farm to create a separate 2nd farm.

I'm happy to help if needed.

@Techwriter40 Techwriter40 merged commit 0a76fb8 into MicrosoftDocs:live Oct 4, 2018
@Yvand
Copy link
Contributor Author

Yvand commented Oct 8, 2018

Wow, this was approved really fast.
Now that the article is live I see some issues with the links and a typo in the script.
I also see improvements to make in the text and in the prereqs. I'll submit a new PR very soon to correct all of this.

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

Successfully merging this pull request may close these issues.

4 participants