My guess is that I’m doing something wrong with that qrexec policies in Dom0. I noticed that those new policies are somehow killing the copy/paste across VMs. When they are active, the copy/paste is not working. When I remove them it is working again.
Strange but interesting. I want to give a try to “split-everything” “socketable” (smartcard, keepassxc, whatever) and possibly protonmail, will tell you the results.
Well, that syntax was correct at the time that post was published. Even if the syntax was subsequently changed, I’m not sure it makes sense to “retcon” that change back into an old post, as that would make the post inaccurate relative to the state of the software at the time at which it was published. If you think about it, taking this to its logical conclusion would require us to constantly update all old posts as the system changes, which wouldn’t really make sense. It’s a post with a specific date, so it’s supposed to be accurate as of that date. It’s not a documentation page that’s supposed to be accurate as of a specific release or as of the present time. If we’re missing a documentation page for this and just leaning on the old post in the absence of proper documentation, then please open an issue to add the missing documentation (or contribute it).
However, do note that the rest of the article clearly indicates a space character is the proper syntax. No judgment, but I can fully understand why the sample line would be used when trying to simply make things work. I’d vote for updating the article only because AFAIK, this article is the de facto documentation for 4.1 syntax.
Yes, I fully agree with that. However this is a different situation, where this comma was wrong even at the time of posting the article (as @icequbes1 already indicated).
The article is liked in the 4.1 RC1 news article, saying in the line above that it explains the new features in depth. Seeing this, I also felt that this is the defacto documentation of this new syntax.
Considering the importance of this change in Qrexec and the prominent placement of this article in the recent announcements, I guess it would help also other users if this typo would be corrected (e.g. by correcting it and adding something like “Edit: Typo corrected in the example of the ConnectTCP policy” to the article).
I surely would not expect that every old news article gets updated whenever a feature/syntax changes later
Oh, well of course that’s a different story! Typo fixed:
In the future, please feel free to submit typo fix PRs yourself. Anyone can do it!
As I said above:
I don’t think it’s necessary to add this to the body content of an article just because a typo was fixed. That’s exactly why we have the Git commit log.
