XMPP Council - 2018-04-19

  1. SamWhited has joined

  2. Tobias has joined

  3. SamWhited has left

  4. Zash has left

  5. guus.der.kinderen has left

  6. guus.der.kinderen has left

  7. Dave has left

  8. guus.der.kinderen has left

  9. dwd has left

  10. guus.der.kinderen has left

  11. guus.der.kinderen has left

  12. SamWhited has joined

  13. guus.der.kinderen has left

  14. SamWhited has left

  15. guus.der.kinderen has left

  16. peter has left

  17. SamWhited has joined

  18. guus.der.kinderen has left

  19. guus.der.kinderen has left

  20. guus.der.kinderen has left

  21. guus.der.kinderen has left

  22. guus.der.kinderen has left

  23. SamWhited has left

  24. SamWhited has joined

  25. guus.der.kinderen has left

  26. SamWhited has left

  27. guus.der.kinderen has left

  28. dwd has joined

  29. guus.der.kinderen has left

  30. guus.der.kinderen has left

  31. guus.der.kinderen has left

  32. peter has joined

  33. SamWhited has joined

  34. Tobias has joined

  35. guus.der.kinderen has left

  36. guus.der.kinderen has left

  37. SamWhited has left

  38. guus.der.kinderen has left

  39. guus.der.kinderen has left

  40. guus.der.kinderen has left

  41. guus.der.kinderen has left

  42. guus.der.kinderen has left

  43. Dave has left

  44. dwd has left

  45. SamWhited has joined

  46. Dave has left

  47. guus.der.kinderen has left

  48. guus.der.kinderen has left

  49. SamWhited has left

  50. guus.der.kinderen has left

  51. guus.der.kinderen has left

  52. moparisthebest has left

  53. guus.der.kinderen has left

  54. guus.der.kinderen has left

  55. guus.der.kinderen has left

  56. guus.der.kinderen has left

  57. SamWhited has joined

  58. guus.der.kinderen has left

  59. guus.der.kinderen has left

  60. guus.der.kinderen has left

  61. SamWhited has left

  62. guus.der.kinderen has left

  63. guus.der.kinderen has left

  64. guus.der.kinderen has left

  65. guus.der.kinderen has left

  66. SamWhited has joined

  67. peter has left

  68. Tobias has left

  69. SamWhited has left

  70. SamWhited has joined

  71. guus.der.kinderen has left

  72. guus.der.kinderen has left

  73. guus.der.kinderen has left

  74. guus.der.kinderen has left

  75. SamWhited has left

  76. guus.der.kinderen has left

  77. guus.der.kinderen has left

  78. guus.der.kinderen has left

  79. guus.der.kinderen has left

  80. guus.der.kinderen has left

  81. Syndace has left

  82. Syndace has joined

  83. guus.der.kinderen has left

  84. guus.der.kinderen has left

  85. jere has joined

  86. moparisthebest has left

  87. guus.der.kinderen has left

  88. guus.der.kinderen has left

  89. guus.der.kinderen has left

  90. guus.der.kinderen has left

  91. SamWhited has left

  92. jere has joined

  93. ralphm has joined

  94. ralphm has left

  95. dwd has joined

  96. dwd has left

  97. ralphm has joined

  98. Dave has left

  99. Dave has left

  100. Dave has left

  101. dwd has joined

  102. dwd has left

  103. Tobias has joined

  104. ralphm has left

  105. ralphm has joined

  106. Dave has left

  107. Dave has left

  108. flow

    Dave, right, that is at least misleading if not a contradiction. But I'm not sure if my conclusion would be to get rid of the 'execute' action all together. My approach would be to make it an always possible action, in order to improve the usability of xep50.

  109. SamWhited has left

  110. guus.der.kinderen has left

  111. SamWhited has joined

  112. dwd has joined

  113. guus.der.kinderen has left

  114. guus.der.kinderen has left

  115. SamWhited has left

  116. SamWhited has joined

  117. dwd has left

  118. Dave has left

  119. Dave has left

  120. Dave has left

  121. guus.der.kinderen has left

  122. Dave has left

  123. vanitasvitae has left

  124. guus.der.kinderen has left

  125. Dave has left

  126. Dave has left

  127. Dave has left

  128. Dave has left

  129. vanitasvitae has joined

  130. Dave has left

  131. Dave has left

  132. ralphm has left

  133. Dave has left

  134. Kev

    The thing is, execute's entirely redundant and as-specced is broken.

  135. Kev

    So while we could fix it, it would still be redundant.

  136. dwd has joined

  137. Dave has left

  138. Dave has left

  139. guus.der.kinderen has left

  140. moparisthebest has left

  141. Dave has left

  142. moparisthebest has joined

  143. flow

    If you assume 'execute' to be always 'next', then yes, it is entirely redundant. If you make it a "smart" action which always points to the next likely action (which is what my PR does), then it is IMHO no longer redundant and a worthwhile feature

  144. guus.der.kinderen has left

  145. Kev

    It's still redundant, because it's still an extra action that doesn't do anything different to the existing actions.

  146. flow

    We probably have different definitions of "redundant"

  147. Dave has left

  148. Dave has left

  149. Ge0rG

    Maybe it's just an awkward and misleading way to tell the client which action is the default one?

  150. dwd has left

  151. dwd has joined

  152. flow

    Ge0rG, right now, I'm more concerned with the action, not the attribute

  153. Kev

    Ge0rG: We have the mechanism for specifying the default already, the execute action doesn't do that.

  154. Dave has left

  155. dwd has left

  156. dwd has joined

  157. Dave has left

  158. Ge0rG

    So the execute attribute is the default next action, and the execute action is... some always-present default action that needs to be injected by the client?

  159. Dave has left

  160. dwd has left

  161. flow

    Ge0rG, xep50 makes multiple statements about the execute attribute and action. Let me wikify it for your, one moment please.

  162. Ge0rG

    flow: I've skimmed through the xep now, thanks

  163. Dave has left

  164. dwd has joined

  165. Dave has left

  166. flow

    Ge0rG, I think it comes down to the statements "The "execute" attribute specifies what the action "execute" is equivalent to." (in 4.2) and "The action "execute" is always allowed, and is equivalent to the action "next"." (in 3.4) contradicting each other.

  167. Dave has left

  168. flow

    See also https://wiki.xmpp.org/web/XEP-Remarks/XEP-0050:Ad-Hoc_Commands

  169. dwd has left

  170. Dave has left

  171. dwd has left

  172. Dave has left

  173. Dave has left

  174. dwd has joined

  175. Remko has joined

  176. Dave has left

  177. Dave has left

  178. Dave has left

  179. flow

    So I've pushed a small change to my PR: https://github.com/xsf/xeps/pull/591

  180. flow

    Now the only difference between Kev's PR and mine is that Kev allows "execute" to be an invalid action, whereas I specify a sane fallback so that you can simply always use "execute" as action.

  181. dwd has left

  182. jonasw

    maybe take that discussion to xsf@?

  183. Dave

    So I don't see the utility of an execute action given we always know what the default action presented in the UI should be.

  184. Ge0rG

    maybe take that discussion to standards@?

  185. jonasw

    or even that

  186. Ge0rG

    Dave: I agree. Conceptually, the execute action is redundant and should best be removed.

  187. Ge0rG

    However, I have no idea about the installed base and how it would be affected

  188. jonasw

    deprecating the use and straightening the definition would be the best course of action then?

  189. Dave

    Ge0rG, Sure, but deprecating it seems the action here.

  190. dwd has joined

  191. Dave has left

  192. dwd has left

  193. Kev

    Ge0rG: Given no-one understands the execute command at the moment, including the author, given the text, I think it's reasonable to believe the installed base is broken. Particularly as this was what caused the thread in the first place.

  194. Dave has left

  195. Ge0rG

    Kev: this is just an assumption, and I'm hesitant to make changes to a Draft XEP that might break even more things.

  196. jonasw looks at aioxmpps Ad-Hoc implementation

  197. jonasw

    it seems I happily operate on the basis of "Execute is a sane default for the action to do because it’s always allowed" and "the using protocol will know what’s needed better than the base library does".

  198. Kev

    jonasw: Which is broken, because Execute isn't always allowed, because sometimes it's Next, which doesn't exist ;)

  199. Dave has left

  200. jonasw

    and the example UI I made doesn’t care about EXECUTE at all and only shows the allowed subset of cancel, next, prev and complete buttons

  201. jonasw

    (sorry for caps, that’s the constant name)

  202. Kev

    Ge0rG: There are three options: 1) Leave everything broken 2) Fix it by specifying the redundant action (possibly breaking already broken implementations) 3) Fix it by deprecating execute (possibly breaking already broken implementations)

  203. Kev

    I think we may as well go for the conceptually Right solution here, which is (3).

  204. Kev

    jonasw: Yes, that's the sane UI. For anything UI based I can't imagine why you would ever send execute for a multi-action form.

  205. jonasw

    I’m all in for (3) by the way :)

  206. jonasw

    although, remember that the execute action is used to *start* an Ad-Hoc session :)

  207. Ge0rG

    Does (3) imply that the service always must provide at least one <action>?

  208. Dave has left

  209. jonasw

    Ge0rG, since "cancel" is implied, no

  210. jonasw

    but it would make sense for a service to prefer status="completed" + an (error) message over not supplying any actions.

  211. flow

    So (3) would mean "execute" becomes a special case action that can only be used to initiate a command?

  212. flow

    Note that we could possibly also do (2) and (3)

  213. Dave has left

  214. dwd has joined

  215. Zash has left

  216. guus.der.kinderen has left

  217. Kev

    > So (3) would mean "execute" becomes a special case action that can only be used to initiate a command? I think it does.

  218. Zash has left

  219. guus.der.kinderen has left

  220. guus.der.kinderen has left

  221. Dave has left

  222. dwd has left

  223. Dave has left

  224. Dave has left

  225. Dave has left

  226. Dave has left

  227. Dave has left

  228. Dave has left

  229. guus.der.kinderen has left

  230. Zash has left

  231. vanitasvitae has left

  232. dwd has joined

  233. dwd has left

  234. guus.der.kinderen has left

  235. guus.der.kinderen has left

  236. guus.der.kinderen has left

  237. guus.der.kinderen has left

  238. guus.der.kinderen has left

  239. guus.der.kinderen has left

  240. ralphm has left

  241. Dave has left

  242. Dave has left

  243. Zash has left

  244. Dave has left

  245. Dave has left

  246. Dave has left

  247. Dave has left

  248. moparisthebest has joined

  249. moparisthebest has joined

  250. dwd has joined

  251. dwd has left

  252. guus.der.kinderen has left

  253. vanitasvitae has left

  254. vanitasvitae has joined

  255. guus.der.kinderen has left

  256. guus.der.kinderen has left

  257. vanitasvitae has left

  258. ralphm has joined

  259. vanitasvitae has joined

  260. Remko has left

  261. Remko has joined

  262. ralphm has joined

  263. Kev has left

  264. guus.der.kinderen has left

  265. guus.der.kinderen has left

  266. guus.der.kinderen has left

  267. daniel has joined

  268. Zash has left

  269. Zash has left

  270. ralphm has left

  271. Dave has left

  272. dwd has joined

  273. ralphm has left

  274. ralphm has joined

  275. guus.der.kinderen has left

  276. guus.der.kinderen has left

  277. jere has joined

  278. ralphm has left

  279. guus.der.kinderen has left

  280. Zash has left

  281. ralphm has left

  282. daniel has left

  283. Zash has left

  284. Zash has left

  285. Zash has left

  286. Dave has left

  287. Zash has left

  288. moparisthebest has left

  289. dwd has left

  290. SamWhited has left

  291. Kev has left

  292. SamWhited has joined

  293. dwd has left

  294. Tobias has left

  295. Tobias has left

  296. guus.der.kinderen has left

  297. SamWhited has left

  298. Kev has joined

  299. guus.der.kinderen has left

  300. SamWhited has left

  301. ralphm has joined

  302. Dave has left

  303. dwd has left

  304. guus.der.kinderen has left

  305. guus.der.kinderen has left

  306. ralphm has left

  307. SamWhited has left

  308. peter has joined

  309. SamWhited has left

  310. guus.der.kinderen has left

  311. guus.der.kinderen has left

  312. Dave has left

  313. dwd has left

  314. dwd has left

  315. SamWhited has left

  316. Tobias has left

  317. ralphm has joined

  318. Tobias has joined

  319. guus.der.kinderen has left

  320. dwd has left

  321. guus.der.kinderen has left

  322. Tobias has joined

  323. Tobias has joined

  324. SamWhited has left

  325. Dave has left

  326. dwd has joined

  327. dwd has left

  328. Dave has left

  329. dwd has joined

  330. Dave has left

  331. dwd has left

  332. dwd has joined

  333. SamWhited has left

  334. dwd has left

  335. Dave has left

  336. dwd has joined

  337. dwd has left

  338. Dave has left

  339. dwd has joined

  340. dwd has left

  341. Dave has left

  342. dwd has joined

  343. dwd has left

  344. dwd has joined

  345. dwd has left

  346. SamWhited has left

  347. Dave has left

  348. dwd has joined

  349. SamWhited has left

  350. Dave has left

  351. dwd has left

  352. dwd has joined

  353. Dave has left

  354. jere has left

  355. jere has joined

  356. jere has left

  357. jere has joined

  358. SamWhited has left

  359. dwd has left

  360. Dave has left

  361. Dave has left

  362. dwd has joined

  363. dwd has left

  364. dwd has joined

  365. SamWhited has left

  366. Dave has left

  367. dwd has left

  368. dwd has joined

  369. moparisthebest has joined

  370. dwd has left

  371. Dave has left

  372. Dave has left

  373. ralphm has left

  374. dwd has joined

  375. dwd has left

  376. dwd has joined

  377. dwd has left

  378. SamWhited has left

  379. guus.der.kinderen has left

  380. dwd has joined

  381. guus.der.kinderen has left

  382. ralphm has joined

  383. SamWhited has left

  384. Kev has left

  385. guus.der.kinderen has left

  386. guus.der.kinderen has left

  387. Dave has left

  388. peter has left

  389. ralphm has left

  390. dwd has left

  391. Dave has left

  392. guus.der.kinderen has left

  393. Dave has left

  394. SamWhited has left

  395. Dave has left

  396. Dave has left

  397. Dave has left

  398. Dave has left

  399. Dave has left

  400. Dave has left

  401. Dave has left

  402. Dave has left

  403. guus.der.kinderen has left

  404. Dave has left

  405. Dave has left

  406. SamWhited has left

  407. guus.der.kinderen has left

  408. ralphm has joined

  409. guus.der.kinderen has left

  410. Dave has left

  411. Dave has left

  412. Dave has left

  413. Dave has left

  414. Dave has left

  415. ralphm has left

  416. dwd has joined

  417. guus.der.kinderen has left

  418. SamWhited has left

  419. Dave has left

  420. dwd has left

  421. Dave has left

  422. Dave has left

  423. Dave has left

  424. dwd has joined

  425. guus.der.kinderen has left

  426. dwd has left

  427. guus.der.kinderen has left

  428. Dave has left

  429. SamWhited has left

  430. Dave has left

  431. dwd has joined

  432. dwd has left

  433. dwd has joined

  434. dwd has left

  435. Dave has left

  436. dwd has joined

  437. SamWhited has left

  438. guus.der.kinderen has left

  439. SamWhited has left

  440. SamWhited has joined

  441. ralphm has joined

  442. guus.der.kinderen has left

  443. SamWhited has left

  444. SamWhited has joined

  445. guus.der.kinderen has left

  446. Dave has left

  447. dwd has left

  448. dwd has joined

  449. Dave has left

  450. SamWhited has left

  451. dwd has left

  452. dwd has joined

  453. moparisthebest has joined

  454. dwd has left

  455. dwd has joined

  456. SamWhited has joined

  457. Kev has joined

  458. dwd has left

  459. Remko has left

  460. SamWhited has left

  461. peter has joined

  462. SamWhited has joined

  463. SamWhited has left

  464. Zash has left

  465. Dave has left

  466. Zash has left

  467. jere has joined

  468. SamWhited has joined

  469. Zash has joined

  470. jere has joined

  471. Dave has left

  472. Dave has left

  473. guus.der.kinderen has left

  474. pep. has left

  475. Dave has left

  476. Dave has left

  477. Dave has left

  478. guus.der.kinderen has left

  479. Dave has left

  480. Dave has left

  481. Dave has left

  482. dwd has joined

  483. Dave has left

  484. dwd has left

  485. Dave has left

  486. SamWhited has left

  487. SamWhited has joined

  488. dwd has joined

  489. SamWhited has left

  490. SamWhited has joined

  491. SamWhited has left

  492. SamWhited has left

  493. guus.der.kinderen has left

  494. SamWhited has left

  495. Dave has left

  496. dwd has left

  497. Dave has left

  498. dwd has joined

  499. SamWhited has left

  500. dwd has left

  501. guus.der.kinderen has left

  502. SouL has joined

  503. ralphm has left

  504. SamWhited has left

  505. Dave has left

  506. dwd has joined

  507. dwd has left

  508. dwd has joined

  509. guus.der.kinderen has left

  510. SamWhited has left

  511. dwd has left

  512. SamWhited has left

  513. SamWhited has left

  514. SamWhited has joined