<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>335668</bug_id>
          
          <creation_ts>2014-06-01 23:13:57 +0000</creation_ts>
          <short_desc>Dialog showing plaintext password while opening a remote (ftp) file in gnome application</short_desc>
          <delta_ts>2018-05-06 00:15:31 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>krusader</product>
          <component>net-connection</component>
          <version>2.4.0-beta3 &quot;Single Step&quot;</version>
          <rep_platform>Ubuntu</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CONFIRMED</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>HI</priority>
          <bug_severity>critical</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>371796</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter>simone.tedeschini</reporter>
          <assigned_to name="Krusader Bugs Distribution List">krusader-bugs-null</assigned_to>
          <cc>alex.bikadorov</cc>
    
    <cc>clearmartin</cc>
    
    <cc>jan_lepper</cc>
    
    <cc>krusader-bugs-null</cc>
    
    <cc>walch.martin</cc>
          
          <cf_commitlink></cf_commitlink>
          <cf_versionfixedin></cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1452555</commentid>
    <comment_count>0</comment_count>
    <who name="">simone.tedeschini</who>
    <bug_when>2014-06-01 23:13:57 +0000</bug_when>
    <thetext>Trying to open a remote file from a ftp server in gnome applications (e.g. pdfs in evince, txts in gedit, etc.), krusader launches the application through this command (seen in ps output):
&gt;&gt;&gt; /usr/bin/gedit ftp://user:{{password}@172.17.2.1:21/folder/file.txt
where {{password}} is actually the password I previously entered in krusader to open the connection

Then the classical gnome password dialog appears saying:
&quot;Enter password for user:{{password}@172.17.2.1:21&quot;
displaying the password

Reproducible: Always

Steps to Reproduce:
1.Connect to a pasword-protected ftp server
2.Open a file using a gnome application
3.
Actual Results:  
Gnome password dialog shown the password in plaintext

Expected Results:  
Don&apos;t show the password (don&apos;t include it in the command given to open the application)

System environment:
Ubuntu 13.04
Gnome 3.10.4</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1452651</commentid>
    <comment_count>1</comment_count>
    <who name="">simone.tedeschini</who>
    <bug_when>2014-06-02 09:45:54 +0000</bug_when>
    <thetext>EDIT:
It&apos;s Ubuntu 14.04, sorry</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1549560</commentid>
    <comment_count>2</comment_count>
    <who name="Jan Lepper">jan_lepper</who>
    <bug_when>2015-10-10 13:07:44 +0000</bug_when>
    <thetext>Note to self and other developers:
This is most likeley caused by not using KUrl::prettyUrl() / QUrl::toDisplayString() which remove the password from the url string.
Needs separate fix in kde4 branch and master.

Places to check for right usage of KUrl/QUrl:
Panel/panelfunc.cpp
UserAction/expander.cpp
and probably others !</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1582243</commentid>
    <comment_count>3</comment_count>
    <who name="Martin Kostolný">clearmartin</who>
    <bug_when>2016-03-13 12:36:45 +0000</bug_when>
    <thetext>Hi! I cannot replicate this issue on krusader-git. @simone.ted can you confirm that it is fixed?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1635131</commentid>
    <comment_count>4</comment_count>
    <who name="Alex Bikadorov">alex.bikadorov</who>
    <bug_when>2016-10-24 19:55:30 +0000</bug_when>
    <thetext>Still in master. Must be fixed!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1635357</commentid>
    <comment_count>5</comment_count>
    <who name="Alex Bikadorov">alex.bikadorov</who>
    <bug_when>2016-10-25 15:56:52 +0000</bug_when>
    <thetext>The problem is actually everywhere. If the initial ftp connection is made by writing the password in plain text into the navigation bar (&quot;ftp://user:password@server.com&quot;) the password is never removed. 
* VFS uses the URL with the password while navigating  on the ftp server
* all file URLs contain the password, it is not removed when opening files or copying them to clipboard
* sometimes debug output with the URL and password is printed

Dolphin has exactly the same problems.

Removing the password from all URLs is not that simple cause the password is never saved and browsing wont work after the first directory listing without entering the password again.

The best solution would be if the KIO listing job saves the password internally and it can be removed after the first connection. This is currently done anyway if no password is provided and the authentication dialog appears.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1636131</commentid>
    <comment_count>6</comment_count>
    <who name="Alex Bikadorov">alex.bikadorov</who>
    <bug_when>2016-10-28 19:12:32 +0000</bug_when>
    <thetext>
I don&apos;t think we can solve this on our own.
* either we remove the password from the URL and the user has to enter it again
* or we stay with the current situation leaving the password in plain text everywhere. 

The latter case is highly insecure and we should at least tell the user about it. Maybe some warning dialog appearing or something.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>