<table><tr><td style="">mikhailnov added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18878">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D18878#409822" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18878#409822</a>, <a href="https://phabricator.kde.org/p/sitter/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@sitter</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p><a href="https://blogs.technet.microsoft.com/filecab/2016/09/16/stop-using-smb1/" class="remarkup-link" target="_blank" rel="noreferrer">https://blogs.technet.microsoft.com/filecab/2016/09/16/stop-using-smb1/</a></p>

<p>I cannot approve this. It is opening potential security issues, overrides sysadmin configs, diverges from sound upstream defaults, goes against advise and practises set forth by Microsoft, continues to avoid moving away from a deprecated 30 year old protocol.</p></div>
</blockquote>

<p>Those who want highest security desiable smb1 on server side, not client. Also, don't forget about some old versions of Windows or proprietary NAS. Please llet users decide themselves, what is more important for them.<br />
We don't use SMB1 to connect in this patch, we use it _only_ for listing smb://. If you do</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">smbclient -L localhost</pre></div>

<p>you will see:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">Reconnecting with SMB1 for workgroup listing.</pre></div>

<p>But only smbclient command line utility does it, not libsmbclient.so.0, that's why this patch is needed.</p>

<blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D18878#409931" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18878#409931</a>, <a href="https://phabricator.kde.org/p/sitter/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@sitter</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>No. This is already unnecessary. If a user requires SMB1 it may be tuned via /etc/samba/smb.conf.<br />
 And that tuning would never be necessary if we had ws-discovery support I think... there is no Windows version which doesn't have ws-discovery and (all?) non-windows systems have DNSSD implementations available. So, between ws-discovery and dnssd pretty much all systems would be supported.</p>

<p>Edit: ...with any possible remainder being up to drop down to SMB1 if the system admin finds that necessary</p></div>
</blockquote>

<p>I've made this patch mainly to discover Linux machines in the network, and Samba has no implementation of ws-discovery. External not common software like wsdd <a href="https://github.com/christgau/wsdd/" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/christgau/wsdd/</a> has to be installed (and not properly tested network daemons may be a securitry risk without propper isolation like chroot).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R320 KIO Extras</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18878">https://phabricator.kde.org/D18878</a></div></div><br /><div><strong>To: </strong>mikhailnov, Dolphin, Frameworks, sitter, broulik<br /><strong>Cc: </strong>ngraham, acrouthamel, kde-frameworks-devel, kfm-devel, alexde, feverfew, michaelh, spoorun, navarromorales, firef, andrebarros, bruns, emmanuelp, mikesomov<br /></div>