• Software
• Assessment
• 20121023-S-A-MiniTOP

• To Software Software - To Software-Assessment Software/Assessment - To previous meeting - To next meeting

Minutes of the MiniTOP on the 2012-10-23

Setting

The MiniTOP will be held via telco 22:00 CEST

Attendees: BenBe, Marcus, magu, Uli, Timo, Michael, dirk

Topics

(skip to agenda)

Action items from last meeting Meeting Action Items

Agenda

1. Preface

2. DEV on bug 1023/1054 "Thawte Patch"

• "Thawte points removal, final step" bug #1023

  • bug #1023 Testing (6.php)
• last patch transfered to production system 2012-05-30
• what are the next steps for thawte points revoke?
  • points settings codes eg 50 pts open gpg/pgp, which certs avail by how many pts
  • 15.php needs rename to 10.php

• next step in: bug #1054 Review the code regarding the new point calculation in ./includes/general.php (current state: testing)

  • email debug notification, search for other solution

  • testing scenarios: see bug note c3163

    • some explanations
  • assure someone - f2f, ttp, the sentence "Only tick the next box if the Assurance was face to face" is conflicting, fixed
  • new patches by dirk, pushed to cacert-devel, (update 2012-09-18)
  • tverify removed (?)

  • merge conflict with account id 60 (eg email removal), see bug #823

  • max_points() routine replaced by new max_points() routine
  • get_assurer_status(), output_summary_content() with parameter 0 replaced by max_points()
  • received_points()

3. 2nd review of about again 5 remaining patches

• Software-Assessors task

• Benny pre-views done

  • neo

    bug #978 Invalid SPKAC requests are not properly validated

    recheck full certs signing procedures duplicate report to bug#540

    5 {0}

  • from meeting 2012-07-17:
    • 5 patches reviewed
    • 3 simple, bugs 540 (fixed), 789 (fixed), 981 (reviewed)
    • 2 with some difficultys, 978 (related to bug#540), complexest one: 1024 (reviewed)

• bug #978 bug 978 (weak keys) (bug 918)

  • invalid key format, no regular error message, something wrong, error code # identified
  • debugging infos from user + infos from critical team with error code #, was spkac routine
  • one test done 2011-12-17 by JensK

  • uli, marcus: more tests: certs routine, weak keys (small keys test), relates to bug#540 tests

  • (week 7)

  • 2012-10-16: bug #540 no new findings regarding iOS5 problem

• bug#1004, stats, Marcus + Uli did some tests, one problem identified, fixed 2012-08-25 by NEO

  • fully re-tested by 2: 2012-08-25 (at froscon)

  gagern, neo	bug #440 Problem with subjectAltName	tested, needs 2nd review, rejected, new deployment getcn/getalt procedure	{r} 2
  neo	bug #978 Invalid SPKAC requests are not properly validated	recheck full certs signing procedures duplicate report to bug#540	5 {0}
  neo	bug #1004 Stats page improvement	tested by 2, needs 2nd review	{0}
  neo	bug #860 someone accessed your password and secret questions notification	tested by 2, needs 2nd review	{0}
  neo	bug #922 missing "certificate about to expire" messages	tested, reviewed by 2, needs 2nd review	{0}

4. Patches Overview - Testing

1. bug #835 CATS test on testserver http://cats1.it-sls.de/

   • create client cert

   • go over to http://cats1.it-sls.de/ pass a cats test

   • inform Ted to trigger a transfer of the tests to the testserver
   • check if CATS test passed to testserver
   • test with different accounts
     1. members age GT 18

        1. member < 100 pts, pass the CATS test

        2. member >= 100 pts, pass the CATS test

     2. members age GT 14 and LT 18

        1. member < 100 pts, pass the CATS test

        2. member >= 100 pts, pass the CATS test

     3. members age LT 14

        1. member < 100 pts, pass the CATS test

        2. member >= 100 pts, pass the CATS test

   • finish and report the tests, no need to transfer to production

2. bug #1054, test 1054.3.6, bug #1035

   • create several types of certs (client certs, server certs, org client certs, org server certs) and analyse the content of the certs -> subjectAltName and CN with single SAN and multiple SANs

   • renew the certs

   • addtl. tests ? Marcus? Magu? BenBe?

   • 2012-10-02 dirk: problems with git push #1054, got fixed
   • DEV on bug 1023/1054 "Thawte Patch"
     • check last changes by dirk to transfer into test scenarios

     • Bug #1054 test scenarios

   • see reference notes note 3225 on bug #1101 and note 3245 on bug #1101

3. bug #922 missing "certificate about to expire" messages

   • you can use previous test to also check "certificate about to expire" messages
   • notification expected: 1d, 15d, 30d, 45d
   • Uli: Marcus plz test again
   • Marcus+Uli: plz add serno of cert about to expire into the message text
   • NEO: added serno on Oct 2nd
   • Uli: 15d notification rcvd at 5th, 6th Oct, last 1d expiry warning expected: Oct 19

4. bug #964 and bug #1017 , relates also to bug #1054, test 1054.3.6 - Chrome certificate enrollement (relates to #964 "Black Jack")

   • create client certs, go to signing routine
   • new routine with 3 different potential signed public key download routines /account.php?id=6 list 3 options
     1. Install the certificate into your browser (tested)
     2. Download the certificate in PEM format
     3. Download the certificate in DER format

   • bug #1017 Chrome certificate enrollement

     • BenBe will pickup

     • bug #1017, doing some more tests?

       • new routine with 3 different potential signed public key download routines /account.php?id=6 list 3 options
         1. Install the certificate into your browser (tested)
         2. Download the certificate in PEM format
         3. Download the certificate in DER format
       • Alex, Marcus doing some more tests

5. bug #1054 (Thawte patch) tests passed ?

6. Marcus Bugs list

   • see Software/BugsOverview

   • according to Bugs # 976

     • 0000976: List of update request for webdb database structure upgrade with tables / fields

     • addtl_notes table hasn't been added in patch bug 976 on 2011-11-25

7. new bug #1095 "Problems with creating server sertificate where the csr is created with Java SDK Tools"

   • cmdline sample: keytool -genkey -alias test.test.net -keyalg RSA -keystore test.test.net.ks -validity 1095
   • NEO couldn't reproduce the problem using keytool, tested against production and testserver
   • identified as weak key usage: csr used MD2 encryption, not or no longer supported by openssl, add new error message

8. bug #440, bug #1101 (extract CSR) (back under development)

   • ASN.1 format
   • CSR extract: needed for signing: email address, hostname
   • Timo will write a CSR parser
   • Current:
     • CN will be parsed
     • some information about public key
   • ASN.1 php library
   • Whats about UTF-8 ?
   • IDN's

     • Policy: p20091108 CPS to drop assurer critieria and allow IDN certificates in specified TLD or single script character sets

     • FAQ Privileges

     • CPS 3.1.7

     • Assurance Handbook - Some more Information

       • Code signing and IDN certificates
         If you are an Assurer, you can get certificates signed/issued by CAcert for code signing and IDNs (International Domain Names).
         Due to the increased possibilities for abuse those certificates have additional requirements. The CPS states that this requires Assurer level, which you meet if you are reading this Handbook. However note that as of 20091106, there is a move to reduce these requirements. Watch this space. 

   • current only client and server certs, other options currently not selectable, except Code Signing
     • extensions currently not supported eg jabber

     • bug #530 XMPP extension not present after renewal

     • bug #87 Issuing certificates for Jabber servers/users

   • parameters: domains, current first becomes CN, others SANs
   • rebuild subject routine ... to check
   • Michael: shall we enforce cn from csr?
     • optional?
     • enforce copy cn to SAN

5. New SA candidates and Coders

1. Heino, not yet prepared, needs first contact
2. How to find coders? Experiences from the Gentoo project

   • http://redmonk.com/dberkholz/2012/07/10/how-to-recruit-open-source-contributors/

   • http://www.slideshare.net/dberkholz/lessons-on-recruiting-open-source-contributors-from-the-google-summer-of-code

   • use as blueprint for other recruits?
3. report from last board meeting - topic Arbitration

   • is added to upcoming board meeting 2012-08-19

6. Long Term Projects

1. NEO: "BlackJack" bug #964

   • NEO: "BlackJack" bug #964 testing from last week -> error codes

     • started implementing

   • how does bug #1017 relate to this bug?

     • cert signing routine
     • ie5 ie6 automatic storage of signed key in local keystore
     • doesn't work under vista, win7
     • msi package is to download and import the keys to the local keystore under vista, win7

     • relates to bug #1099 but is quite different

     • neo sent msi package for testing to u60, benbe; test successful passed
2. Marek's sql class project:
   • is working on charset replacement
3. api project, Carsten continues with portal project not waiting for vendor-api to be delivered
   • vendor-api delayed
     • no coders
     • other projects
     • related to sql class project
   • portal project continues with a workaround, needs an assurer
     • arbitration case on locations database orders outsourcing of find-an-assurer asap
     • with portal function, update of data is possible vs. update of data on critical system is difficult (keep data current for assurers)
     • relation to location database
       1. website find an assurer
       2. scripted mailing for ATE invitations
     • user check that data is still valid eg every 1 year
       • notification at login upto 6 months not online
       • notification by email if not logged in within last 6 months
4. Automated testing system
   • Timo: Unit-test testsystem, phpunit jenkins

   • http://ci.partkeepr.org/job/PartKeepr/

   • https://github.com/NEOatNHNG/cacert-frontendtests

   • can we merge both environments? frontend tests and unit tests?
   • Timo: automated testing systems are mergable
     • frontend test: java, may become a problem, alternate php version?
     • focus on unittests
       • dirk: code or screen?
       • code and screen
     • frontend and unit tests on one machine?
     • trial: port frontend tests
5. Timo: monitoring signer, not yet done
   • Probably Wytze monitors the systems externaly ?!?

   • see Systems overview

   • monitoring system eg Zabbix instead of Nagios?
   • BenBE: Icinga as alternate?
   • Zabbix agents: requires to be the same revision as server
6. Timo, Benny: Distro needs upgrade
   • lenny - support ended Feb 2012
   • upgrade etch to lenny was a long running project
   • squeeze (current stable release) - tests started by critical team
     • "wheezy close before release date
   • Michael: email sent 2012-10-09 regarding squeeze upgrade to critical team
     • response received
     • testing WIP
     • move to sun2 proposed

7. bug #1101 refactoring getalt getcn (Timo)

   • might 1101 comment c3225

   • tries to build a php library for openssl parsing replacement
     1. asn.1 parsing, own library
     2. ???
   • openssl does escaping (per man page) (input? output?)
   • library test thru unit tests
   • openssl command for multiple san's ?
     • undocumented feature?

     • currently only known with -extfile creating-a-certificate-with-multiple-hostnames

7. next meeting

• Tuesday, October 30, 2012 22:00 CEST

Minutes

1. Preface

   1. BenBe: 922 2nd review, currently busy, feels not ready to review this patch

   2. asn1 parse procedure, http://lapo.it/asn1js/

      • getcn, getalt procedure

      • docs für extractit() und getcn(): general.php line.230

      • felicitus: how someone get "CN" from "commonName"? where is it documented that "CN" is "commonName"?
      • OID of commonName is 2.5.4.3, but there is nothing about "CN"
        • BenBE: see Header of OpenSSL-Header
   3. Marcus: OA sql query procedure, NEO to test on testserver

      • relates to a20110118.1 intermediate ruling Part IV.2

      • also bug #512

2. OU info from Org cert not stored (agenda 4.6)

   • addtl_notes table hasn't been added in patch bug 976 on 2011-11-25

   • extend org certs table ? new bug?
   • OU in subject?
   • includes/account.php (17)
   • in org certs it is in subject
   • addtl. field ou ? new bug# ?
   • used bug #1010
3. agenda 3 - patches needs 2nd review

   • bug #978 Invalid SPKAC requests are not properly validated

     • recheck full certs signing procedures
       duplicate report to bug#540

     • routines that checks for weak keys
     • new in lib runcommand()
       • cleanup in other functions

     • -> BenBe, assigned

   • bug #1004 Stats page improvement

     • tested by 2, needs 2nd review
     • moved out to cron job routine

     • -> BenBe, assigned

   • bug #860 someone accessed your password and secret questions notification

     • tested by 2, needs 2nd review
     • strings to translate, gettext combined

     • -> BenBe, assigned

   • bug #922 missing "certificate about to expire" messages

     • you can use previous test to also check "certificate about to expire" messages
     • notification expected: 1d, 15d, 30d, 45d
     • Uli: Marcus plz test again
     • Marcus+Uli: plz add serno of cert about to expire into the message text
     • NEO: added serno on Oct 2nd
     • Uli: 15d notification rcvd at 5th, 6th Oct, last 1d expiry warning expected: Oct 19, passed ok
     • moved to 2nd review

     • tested by 2, needs 2nd review, BenBe passed to other SA

     • -> dirk

4. Patches Testing

   1. bug #835 CATS test on testserver http://cats1.it-sls.de/

      • pass a full cycle on cats1 server to check that info passes to testserver
5. New patches

   • bug #1080 0001080: The link on page to iso code on account.php?id=24 show no result

   • bug #1083 0001083: Resize comment field for adding new organisation administrators

   • BenBe will transfer to testserver

6. next meeting
   • Tuesday, October 30, 2012 22:00 CET (21 UTC)

Fixed Action Items since last or within meeting

• OA sql query procedure, List returned on 2012-10-24

Action Items New

Action items: Meeting Action Items

• CategorySoftwareAssessment