Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

enable guest access for the 3pl/3pid APIs #1986

Merged
merged 2 commits into from
Mar 31, 2017
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 0 additions & 8 deletions synapse/rest/client/v2_alpha/thirdparty.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ def __init__(self, hs):

@defer.inlineCallbacks
def on_GET(self, request):
yield self.auth.get_user_by_req(request)

protocols = yield self.appservice_handler.get_3pe_protocols()
Copy link
Member

Choose a reason for hiding this comment

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

We should keep the auth checks and instead add allow_guest=True param.

defer.returnValue((200, protocols))

Expand All @@ -54,8 +52,6 @@ def __init__(self, hs):

@defer.inlineCallbacks
def on_GET(self, request, protocol):
yield self.auth.get_user_by_req(request)

protocols = yield self.appservice_handler.get_3pe_protocols(
only_protocol=protocol,
)
Expand All @@ -77,8 +73,6 @@ def __init__(self, hs):

@defer.inlineCallbacks
def on_GET(self, request, protocol):
yield self.auth.get_user_by_req(request)

fields = request.args
fields.pop("access_token", None)

Expand All @@ -101,8 +95,6 @@ def __init__(self, hs):

@defer.inlineCallbacks
def on_GET(self, request, protocol):
yield self.auth.get_user_by_req(request)

fields = request.args
fields.pop("access_token", None)

Expand Down