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