XMPP Council - 2024-08-20


  1. daniel

    Agenda for todays meeting just went out. council members please read some backlog so we can come to an agreement on #1365 in todays meeting

  2. daniel

    personal preference: > <p>Note: in older versions of this standard, the inclusion of a &lt;destroy/&gt; element was not explicitly defined to be mandatory (when no alternate location or reason are provided by the room owner). Implementations therefor cannot reliably depend on the existence of this element to identify a room destruction event.</p>

  3. daniel

    this means basically the same text we had last week but with the odd upper case removed

  4. daniel

    normative language for the normative behaviour. non normative text for the note

  5. daniel

    And then we don't have to agree or disagree on if this was clear and normative behavior to begin with. It's just an informal note

  6. moparisthebest

    > normative language for the normative behaviour. non normative text for the note +1 nit: therefore end with e

  7. Kev

    Given that it's required for interop to not rely on it, wouldn't normative s/cannot reliably/MUST NOT/ be appropriate?

  8. daniel

    Arguably not if the normative text is just a clarification and the note points to the existence of some (one?) incompatible implementation

  9. Guus

    I'm fine with that text Daniel. I'd be happy to apply that change after Council finalized discussion. I'm equally happy for you to push that change when you merge the PR - whatever you prefer.

  10. daniel

    My plan is for council to decide on a wording today and then I might as well apply that myself (instead of taking the additional round-trip of asking you, Guus, to modify and vote again next week)

  11. daniel

    Editor me wants a straight forward way to close a bunch of PRs to get them of my plate

  12. Guus

    Why do you think a new vote is needed?

  13. Guus

    In any case, I'm totally fine with that.

  14. daniel

    If it's not needed it should be an easy decision

  15. daniel

    But we didn't have a full vote yet on the current iteration of the PR

  16. daniel

    Plus I yet again changed the wording

  17. Guus

    ah ok

  18. daniel

    It’s time

  19. daniel

    1) Roll call

  20. moparisthebest

    Hello!

  21. daniel

    singpolyma, dan.caseley larma are you around?

  22. singpolyma

    hello

  23. daniel

    2) Agenda bashing

  24. daniel

    nothing to bash

  25. daniel

    3) Editors updates

  26. daniel

    none

  27. daniel

    4) Items for voting

  28. daniel

    a) XEP-0045: Presence sent to occupants of a destroyed room includes a <destroy/> https://github.com/xsf/xeps/pull/1365 i’m not just yet starting voting on that particular PR instead I would like to propose the following note to be used instead > <p>Note: in older versions of this standard, the inclusion of a &lt;destroy/&gt; element was not explicitly defined to be mandatory (when no alternate location or reason are provided by the room owner). Implementations therefor cannot reliably depend on the existence of this element to identify a room destruction event.</p>

  29. daniel

    are council members happy with that? or does any one have a different proposal on how to proceed with that PR

  30. daniel

    (accept as is, reject)

  31. moparisthebest

    s/therefor/therefore/ and then +1 for that wording

  32. singpolyma

    +1

  33. larma

    +1

  34. daniel

    b) XEP-0388: Fix the XML Schema and examples https://github.com/xsf/xeps/pull/1369

  35. larma

    (just IMO, but exact wording doesn't strictly need council to confirm, but can be done by editor as editorial)

  36. singpolyma

    b +1

  37. daniel

    larma, in that case I would have needed the vote for the proposed PR anyway

  38. daniel

    +1

  39. larma

    b) +1

  40. daniel

    moparisthebest?

  41. moparisthebest

    +1

  42. daniel

    c) XEP-0045: Explicitly use bare JIDs when operating on affiliations https://github.com/xsf/xeps/pull/1371

  43. daniel

    +1

  44. singpolyma

    c +1

  45. larma

    +1

  46. moparisthebest

    +1

  47. daniel

    d) XEP-0045: Allow non-owners to retrieve owner and admin lists in non-anonymous rooms https://github.com/xsf/xeps/pull/1372

  48. singpolyma

    d +1

  49. daniel

    +1

  50. moparisthebest

    +1

  51. daniel

    e) XEP-0402: Specify what clients should do if autojoin='0' in bookmark notifications https://github.com/xsf/xeps/pull/1373

  52. daniel

    this is more than a 'clarification'.

  53. daniel

    personally i’m leaning towards +1

  54. daniel

    it seems like the logical opposite to what is explained a sentence prior (join when modified to true)

  55. daniel

    and i certainly implemented it that way

  56. larma

    I disagree

  57. singpolyma

    I don't like this speccing UX stuff

  58. singpolyma

    I understand the xep already sort of does in one place

  59. larma

    How would I signal that I want to join a room only for this session?

  60. larma

    By joining and setting autojoin=0

  61. larma

    I agree that most clients don't want this these days, but it still is a perfectly valid usecase

  62. daniel

    i guess whether or not you consider that UX depends on how you look at that xep entirely

  63. daniel

    i always understood that xep to describe an automated functioniality

  64. daniel

    you can like or dislike that xep

  65. daniel

    but that modification feels in line with the rest

  66. moparisthebest

    I think it's fine, nothing stops a client from allowing you to manually join a room with autojoin 0

  67. moparisthebest

    So I'm +1

  68. daniel

    i think nothing in that xep says that you should even display bookmarks as bookmarks

  69. Zash

    Heh, it has indeed evolved into a secondary roster of sorts :)

  70. moparisthebest

    (fwiw I don't think this is mandating UX, it explains the intent behind values, and the explanation was missing here)

  71. daniel

    so as far as that xep is concerned it's describing only an automated way to sync what mucs you are joined

  72. daniel

    anyway do you want to vote or should I ask the author to bring this to the list first?

  73. larma

    I'm +0

  74. moparisthebest

    Imho not specifying what 0 means is a hole in the spec that needs addressed regardless Since I don't have a better suggestion for addressing it, +1

  75. daniel

    yes. otherwise it feels weird that it mandates that you leave when the bookmark is deleted and when an autojoin is added; but doesn’t say what to do when the autojoin is removed

  76. daniel

    i understand singpolyma point in so far that mandating something in the first place (leave on delete) is a bit weird but that ship has sailed

  77. Zash

    Leave iff you joined due to autojoin=1 ?

  78. daniel

    (in fact in Conversations I only implemented the leave on delete fairly recently after taking a long time to think about it)

  79. daniel

    anyway let me know how you want to proceed; reject, vote on list; open up more list discussion singpolyma

  80. daniel

    we are running up on our 30min time limit

  81. singpolyma

    > yes. otherwise it feels weird that it mandates that you leave when the bookmark is deleted and when an autojoin is added; but doesn’t say what to do when the autojoin is removed I agree this is weird. What is weird IMO is that it says to leave when deleted :P

  82. singpolyma

    xep is stable so I guess there's no way to fix that anyway right?

  83. singpolyma

    I might like it more if it was non-normative instead of a strong SHOULD

  84. larma

    I certainly would be fine with a MAY 😉

  85. dan.caseley

    Sorry, I was in a work thing I couldn't escape. I'm +1 to everything so far. The fact that clients are already treating autojoin=0 this way pushes towards hyperaccuracy of standards, trailing the implementation (perhaps improperly), but if most implementations aren't opinionated on it currently, it seems harmless.

  86. daniel

    fwiw the other join leave instructions are SHOULD too

  87. singpolyma

    > fwiw the other join leave instructions are SHOULD too yes 😞

  88. singpolyma

    Ok. I guess I'll go -0 I don't like it, but the xep is already like this and already stable

  89. daniel

    I don’t know it seems you simply don’t like the (purpose) of the XEP.

  90. larma

    IMO instructing clients to join if autojoin=1 makes sense, because well, the flag means to automatically join. But autojoin=0 does not mean autoleave=1 😉

  91. moparisthebest

    Can always make a new XEP (:

  92. daniel

    it's all SHOULD; so a client wide 'ignore autojoin flags' expert settings can ignore the shoulds

  93. moparisthebest

    > IMO instructing clients to join if autojoin=1 makes sense, because well, the flag means to automatically join. But autojoin=0 does not mean autoleave=1 😉 larma: what does it mean

  94. daniel

    for a 'i want different behaviour on my phone' experience

  95. larma

    Not join automatically, but maybe manually

  96. daniel

    dan.caseley, a-e yes?

  97. dan.caseley

    Yes please

  98. Zash

    inb4 autojoin=complicated client query language expression

  99. daniel

    I’m going +1 on 402 too. which means the XEP passes (with 3 +1 and 2 +0)

  100. daniel

    5) Pending votes

  101. daniel

    none

  102. daniel

    6) Date of Next

  103. moparisthebest

    +1w wfm

  104. daniel

    +1w wfm (but i'll propably be on a hard 30min time limit; shorter is better)

  105. larma

    +1w wfm

  106. singpolyma

    +1w wfm

  107. dan.caseley

    I'll be on holiday (again), but will possibly join all the same

  108. daniel

    7) AOB

  109. moparisthebest

    Thanks Guus for working so hard to polish the rough edges off MUC <3

  110. moparisthebest

    (no other aob here)

  111. daniel

    8) Close

  112. daniel

    thank you all. see you next week

  113. Guus

    It's utterly self serving, but you are welcome

  114. moparisthebest

    Guus: fair but you could have just noted it yourself and not bothered with PRs or discussions so we all appreciate it :)

  115. moparisthebest

    Thanks all!