- 
                Notifications
    
You must be signed in to change notification settings  - Fork 192
 
Description
Bug Description
With DLS enabled, unpublished Sharepoint Online pages are still returned in search queries to all users that have permissions to view the page.
Unpublished pages are ingested and added to the ES index as they are still considered accessible to both owners and editors of the document. However, the Sharepoint Online API doesn't appear to be updating their ACLs for that page when it goes into an unpublished state, so our SPO connector will continue to treat these pages as accessible to anyone with view permissions.
Also, we should consider exposing a field representing the published state of a page (if the Sharepoint API allows for it).
To Reproduce
Steps to reproduce the behavior:
- Add a page to a site in Sharepoint Online and publish it.
 - Within Sharepoint Online, navigate to "Manage access" for the created page and add some users/groups with "View access"
 - Sync content with DLS. Verify that the 
site_pagecreated was ingested and_allow_access_controlcontains the users with view access. - Unpublish the page in Sharepoint Online.
 - Perform a full content sync.
 - Document is still indexed (expected) and 
_allow_access_controlis unchanged (bug). 
Expected behavior
The _allow_access_control field should only contain owners and editors of the page, implying that users performing searches on the index without ownership/edit access of the unpublished page will not see it in their search results