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

XEP-0198: Add section defining SASL2 and BIND2 interaction #1215

Closed
wants to merge 5 commits into from

Conversation

tmolitor-stud-tu
Copy link
Contributor

The title says it all: I've added a section specifying interaction with the upcoming versions of the SASL2 (XEP-0388) and BIND2 (XEP-0386) XEPs.

See #1214 for details about the SASL2 changes.

@horazont
Copy link
Contributor

Even though this is pending the acceptance of #1214, I'm still submitting this to council to judge the general approach of having this in '198 as well as any concerns which may exist under the precondition of #1214 being merged.

@horazont horazont added the Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change. label Oct 12, 2022
@tmolitor-stud-tu
Copy link
Contributor Author

For reference, here is the rendered version: https://dyn.eightysoft.de/final/xep-0198.html#inline

xep-0198.xml Outdated Show resolved Hide resolved
@iNPUTmice
Copy link
Contributor

Hi @tmolitor-stud-tu the feedback @mar-v-in provided hasn’t been addressed yet. This is why council has rejected this for now. But once this minor thing is fixed I can put it back on the agenda.

@tmolitor-stud-tu
Copy link
Contributor Author

@iNPUTmice I'm sorry, I somehow missed that feedback. Should be addressed now :)

</section3>
</section2>
<section2 topic="Inline Stream Resumption" anchor="inline-resume">
<p>To indicate support for inlining stream resumption into the authentication process, the server adds a &lt;resume/> element in the namespace "urn:xmpp:sm:3" to the &lt;inline/> element of SASL2.</p>
Copy link
Contributor

Choose a reason for hiding this comment

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

The example uses a <sm/> element and so do Conversations and prosody

@iNPUTmice
Copy link
Contributor

Merged as #1294

@iNPUTmice iNPUTmice closed this Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants