onesteinbv/addons-onestein

connector_office_365 and scheme replacement

Closed this issue · 2 comments

Hi,

We tested connector_office_365, which works well, although we had to remove this replace:

token = user.office_365_get_token(request.httprequest.url.replace('http', 'https'))

Our URL was already https, so we ended up with httpss:// as scheme. I don't know why you needed this in the first place. If it is required, it could be done with urlsplit and urlunsplit

Thanks, I think it was an artifact of development.

Great, thanks :)