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

Media server sending back its local URL instead of the endpoint URL in confirm requests #70

Open
martin-hewitt opened this issue Jul 8, 2014 · 4 comments
Labels

Comments

@martin-hewitt
Copy link

Related to: buddycloud/buddycloud-http-api#165 (comment)

We're seeing HTTP confirmation stanzas like:

<message type="normal" to="[email protected]/resource" from="mediaserver.example.com" xmlns:stream="http://etherx.jabber.org/streams"><thread>xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx</thread><body>Confirmation message for transaction asdfghjkl</body><confirm id="asdfghjkl" url="http://localhost:8080/[email protected]/asdfghjkl" method="GET" xmlns="http://jabber.org/protocol/http-auth"/></message>

Where the url field is reporting the internal value, not the full API/media server location.

We're using commit 807d71f

@abmargb
Copy link
Collaborator

abmargb commented Jul 8, 2014

Is the http.endpoint property set? As per 64bd569, it should have been used instead of the internal URL.

@lloydwatkin
Copy link
Member

Yes its properly set as we determine our media server endpoint via DISCO (which uses this value).

@abmargb
Copy link
Collaborator

abmargb commented Jul 8, 2014

I'm investigating it then :-)

@abmargb
Copy link
Collaborator

abmargb commented Jul 9, 2014

I've updated mediaserver.buddycloud.org to the latest commit on the master branch. That's what I get from it now:

<message to="[email protected]/resource" type="normal" from="mediaserver.buddycloud.org">
  <thread>xxxxxxxxxxxxxxxxxxxxxxxxxx</thread>
  <body>Confirmation message for transaction w5ln43oy6xm3</body>
  <confirm xmlns="http://jabber.org/protocol/http-auth" url="https://demo.buddycloud.org/api/media_proxy/[email protected]" id="w5ln43oy6xm3" method="GET" />
</message>

The following property is set in mediaserver.properties:

http.endpoint=https://demo.buddycloud.org/api/media_proxy

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

No branches or pull requests

3 participants