Bug 1070171 - (tlssled) Review Request: tlssled - An evaluation tool for SSL/TLS (HTTPS) web server implementations
Review Request: tlssled - An evaluation tool for SSL/TLS (HTTPS) web server i...
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Christopher Meng
Fedora Extras Quality Assurance
:
Depends On:
Blocks: FE-SECLAB
  Show dependency treegraph
 
Reported: 2014-02-26 06:00 EST by Fabian Affolter
Modified: 2014-09-11 11:23 EDT (History)
2 users (show)

See Also:
Fixed In Version: tlssled-1.3-3.fc20, tlssled-1.3-3.fc19
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2014-03-26 10:10:24 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
i: fedora‑review+
limburgher: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Fabian Affolter 2014-02-26 06:00:45 EST
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled-1.3-1.fc20.src.rpm

Project URL: http://www.taddong.com/en/lab.html

Description:
TLSSLed is a Linux shell script whose purpose is to evaluate the security of
a target SSL/TLS (HTTPS) web server implementation. It is based on sslscan, a
thorough SSL/TLS scanner that is based on the openssl library, and on the
"openssl s_client" command line tool. The current tests include checking if
the target supports the SSLv2 protocol, the NULL cipher, weak ciphers based
on their key length (40 or 56 bits), the availability of strong ciphers
(like AES), if the digital certificate is MD5 signed, and the current SSL/TLS
renegotiation capabilities.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=6572387

rpmlint output:
[fab@laptop011 SRPMS]$ rpmlint tlssled-1.3-1.fc20.src.rpm 
tlssled.src: W: spelling-error %description -l en_US sslscan -> scantness
tlssled.src: W: spelling-error %description -l en_US openssl -> slope
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint tlssled-1.3-1.fc20.noarch.rpm 
tlssled.noarch: W: no-documentation
tlssled.noarch: W: no-manual-page-for-binary tlssled
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

Fedora Account System Username: fab
Comment 1 Christopher Meng 2014-02-26 23:34:22 EST
1. #!/usr/bin/env bash

I'm not sure if we need to hack it to /bin/bash or /bin/sh.

2. install with -p.

3. Mix using tab and space:


Requires:	sslscan
Requires:   openssl

Please unify.

------------------------------
Show me the fixed version and I will set +.


PACKAGE APPROVED.
Comment 2 Fabian Affolter 2014-02-27 04:55:54 EST
Thanks for the review, Christopher.

(In reply to Christopher Meng from comment #1)
> 1. #!/usr/bin/env bash
> 
> I'm not sure if we need to hack it to /bin/bash or /bin/sh.

As far as I remember was this topic discussed 4-5 years ago. rpmbuild is picking env up so it will work.

> 2. install with -p.

fixed
 
> 3. Mix using tab and space:

fixed

Updated files:
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled-1.3-2.fc20.src.rpm
Comment 3 Michael Schwendt 2014-02-27 06:03:16 EST
There are mixed feelings about /usr/bin/env.

/usr/bin/env as the dependency is less correct than a strict dependency on /usr/bin/bash (being the bash provided by Fedora). And if someone puts "bash" in a customised $PATH before /usr/bin, this may break the program and/or even make it insecure in case it's a vulnerable old bash.

This also affects other script interpreters, see e.g.
http://fedoraproject.org/wiki/Features/SystemPythonExecutablesUseSystemPython

An old attempt at prohibiting /usr/bin/env in shebang:
https://fedoraproject.org/wiki/Script_Interpreters_%28draft%29
Comment 4 Christopher Meng 2014-02-27 09:44:26 EST
So please use sed to replace the shebang line;

Also, these contain tabs still:

Name:		tlssled
Version:	1.3
Release:	2%{?dist}
Summary:	An evaluation tool for SSL/TLS (HTTPS) web server implementations

License:	GPLv3+
URL:		http://www.taddong.com/en/lab.html
Source:		http://www.taddong.com/tools/TLSSLed_v%{version}.sh
BuildArch:	noarch

Requires:   sslscan
Requires:   openssl
Comment 5 Fabian Affolter 2014-03-04 04:21:01 EST
(In reply to Christopher Meng from comment #4)
> Also, these contain tabs still:

Not sure why rpmlint doesn't pick it up. Anyway, should be fixed now.

* Tue Mar 04 2014 Fabian Affolter <mail@fabian-affolter.ch> - 1.3-3
- Update shebang
- Again spaces

Updated files:
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/tlssled-1.3-3.fc20.src.rpm
Comment 6 Christopher Meng 2014-03-04 23:33:10 EST
PACKAGE APPROVED.
Comment 7 Fabian Affolter 2014-03-05 05:15:01 EST
Thanks again
Comment 8 Fabian Affolter 2014-03-05 05:19:17 EST
New Package SCM Request
=======================
Package Name: tlssled
Short Description: An evaluation tool for SSL/TLS (HTTPS) web server implementations
Owners: fab
Branches: f19 f20
InitialCC:
Comment 9 Gwyn Ciesla 2014-03-06 08:32:20 EST
Git done (by process-git-requests).
Comment 10 Fabian Affolter 2014-09-11 10:16:35 EDT
Package Change Request
======================
Package Name: tlssled
New Branches: el6 epel7
Owners: fab 
InitialCC:
Comment 11 Gwyn Ciesla 2014-09-11 11:23:30 EDT
Git done (by process-git-requests).

Note You need to log in before you can comment on or make changes to this bug.