linuxmint / xapp

Cross-desktop libraries and common resources
GNU Lesser General Public License v3.0
126 stars 44 forks source link

upload-system-info --link-only #154

Open zcot opened 2 years ago

zcot commented 2 years ago

On IRC, it can be maddening, users will paste the entire system info.

A couple of the mods have an ongoing conversation which is better, upload-system-info, or the long form piped with nc. I believe it's the given function, it's really good. And that works perfectly for most users.

But the truth is that there are a notable amount of floods, no matter how much verbiage we can put into the instruction to provide system info, "let us view the web page with you, we only need the link", "please just paste the address, the link of the web page, do not paste more than 1 line", -someone is always going to copy/paste the system info into the channel. It doesn't matter which form, because with upload-system-info, they too easily see the WOT and copy/paste it, but with the piped version, they screw up the pipe and it just spits out the long version instead of the termbin.com link, so again, they see the WOT, and boom!

So, I would love, I guess it seems more simple, just a flag or something, because that's simple for a newbie, -there is no pipe or odd character that might mess someone up. The current upload-system-info script stays intact, because it is required for other functionality other than IRC, but we can get a fool-proofed method for the IRC channels.

Maybe an -L flag? -O for output, or -o for output is going to hang someone up. -l is too closely ambiguous with 1, there's going to be many clarifications about that. Just a test with upload-system-info -fubar (or any other potential malformation) produces a horrendous experience so I vote for anything obvious. But, --link or anything similar I think would be perfect, and that lends to being obvious and even verbose about the command to a user.

My problem is that I only know about 1% of python. :smile:

Another alternative, thrown together by belanthor on the channels, but we were just openly discussing it, he says he doesn't even know python, but after a few minutes played with it and came with the idea of just adding another script entirely (since it would break the functionality for the original script though, called from system reports), so call it upload-system-info-link, or something, but something like that: https://termbin.com/owkc (I see the added lines but I don't know python either).

Is there any chance to get an easy solution on this? :smiley: