[FFmpeg-devel] [PATCH] fftools/ffmpeg_ffplay_ffprobe_cmdutils: add -mask_url to replace the protocol address in the command with the asterisk (*)

Nicolas George george at nsup.org
Mon Dec 19 15:37:10 EET 2022


Marvin Scholz (12022-12-19):
> IIUC this means the `-mask_url` option has to be the first option passed,
> which seems a bit of an unfortunate requirement and is not documented at
> all, as far as I can see. So at least this should be clearly documented
> to prevent users being confused why the get an unrecognised option error
> when they do not pass it as the first option.

Indeed. And I see no reason to have this option processed specially like
that; it requires at least an explanation.

> I am a bit confused how this helps for the issue it tries to solve, as
> for some amount of time, until this is done, it would expose the full
> plaintext URL still, no?

This is unavoidable. Still, having sensitive information visible for a
fraction of a second is better than having sensitive information visible
for the length of a playback or transcoding process.

Regards,

-- 
  Nicolas George


More information about the ffmpeg-devel mailing list