• Software
• Assessment
• 20130205-S-A-MiniTOP

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

Minutes of the MiniTOP on the 2013-02-05

Setting

The MiniTOP will be held via telco 22:00 CET (21:00 UTC)

Attendees: Magu, BenBe, uli, Marcus, 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()
  • Status testing ?
    • debug messages on testserver (1054)
      • test account 1
        • variant 1 (pwd login): points3 (185/100)
        • variant 2 (cert login): points3 (185/100)points4 (185/100)
        • first value relates to wot.php?id=10 count of pts
      • test account 2
        • points3 (350/394)points4 (350/394)
        • 484 AP, anderer weg 64
        • 100
      • test account 3
        • points3 (200/662)
        • problem identified, fix transfered to testserver
• current state: runs in debug mode points1, points2, points3, points4 that reflects conditions
  • conditions points1 and points2 yet undefined

  • meeting 2012-12-11

    • identified points1 + points2 debug points in routine
    • Logged-in, My Details, Edit, change something, submit changes
    • calculate points will be used to select edit mode for name, dob
    • if points==0 edit allowed, otherwise edit prevented
• Patch moved out from testserver
  • new testserver branch stable
    • reason: more and more merge conflicts caused by bug #1054
    • patches to add: 964
    • also: 1070 + 782 (?)

3. 2nd review of remaining patches

• Software-Assessors task

• bug #1119 scheduled visit, not yet announced

• bug #1082 The text on the login form is not shown inside the grey box

  • Hendrik, Inopiae

    Bugs #1082 The text on the login form is not shown inside the grey box

    {0}

  • reviewed by Dirk, ok, needs transfer to production

3.1 Michaels Task 2nd review (10 patches)

• 1121 (see notes under top 4)
• 1122 (see notes under top 4)

  inopiae	bug #1121 Record the CCA acception for the account creation	needs 2nd review	{0}
  inopiae	bug #1122 Give Support the chance to see when the first and the last CCA acception took place	needs 2nd review	{0}

4. Patches Overview - Testing, Development

1. summary - state of patches
   1. 922 needs work (returned from production)
   2. 782 needs work
   3. 440 needs work (NEO) (see also below)
      • Patch bug #440 was defered (timo addtl. work), but this project stalls. What to do with bug #440 ?

        gagern, neo

        bug #440 Problem with subjectAltName

        tested, needs 2nd review, rejected, new deployment getcn/getalt procedure, relates to bug #1101

        {r} 2

   4. 1004 needs work by neo
   5. 1113 needs work by benbe, transfered to cacert-devel
   6. 1017 needs work by neo
   7. 1025 needs testing
   8. copied to cacert-devel, on testserver

      • inopiae	Bugs #1124 Selection of additional languages, sorting is somewhat strange		{0}
        inopiae	Bugs #1112 Exchange the text on the TTP page according to the new TTP programm	live testing, probs with form (if pts < 100)	{0}

        • available 740 fix isn't in compliance to bug #671

        • related bug #1112 is also in question

          inopiae

          bug #740 How to become an assurer is missleading wot 2+4 fixes

          needs testing and 2nd review

          {0}

        • contrary discussion
2. Policy text and Arbitration ruling bug# fixes
   1. Policy text changes

      • new bug #1131 Replace all policies from php to html

        Inopiae

        Bugs #1131 Rename PolicyOnPolicy.php and other Policies too to .html

        {0}

   2. Arbitration ruling text fixes

      1. bug #879

         • CAcert must update the web page on disputes, and include an explanation how to file a dispute (a20091206.1)

3. bug #922 problem, transfered to critical, Wytze did a rollback

   • neo, dirk

     bug #922 missing "certificate about to expire" messages

     tested, reviewed by 2, needs 2nd review

     {0}

   • 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

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

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

   • -> dirk, assigned

   • seems to be ok, ready to go

   • BenBe to transfer to critical team

   • patch transfered, but rolled back. reason: patch brings critical system to hung
     • Analysis
       • testserver less data then production system
       • potential problem distinct clause in query
       • whats about proposals by Timo?

         • proposal 12/11/02 by Timo

       • data count: 1000 on testserver, 900.000 on production
         • create a test set of 900k certs in database?
       • tables used, record counts: domaincerts 74, domlink 75, domains 52
         • which tables, table structure, db format: default myisam
         • domain*, email*, users
         • to contact critical team with general infos about above tables
     • wytze, timo, dirk, benbe, michael discussion by email
       • proposal to wytze, to add indixes (all tables selected by where clauses created, modified, expired, revoke)
       • confirmation by 2nd SA
   • 2013-01-22: still open
   • discuss with NEO
   • database modifications, index modifications
   • performance issues, needs applied into a patch

   • sql query proposal see https://bugs.cacert.org/view.php?id=922#c3347

4. bug #1004 Stats page improvement

   • neo, BenBe

     bug #1004 Stats page improvement

     tested by 2, needs 2nd review

     {0}

   • 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)
   • needs 2nd review
   • moved out to cron job routine

   • -> BenBe, assigned

   • 1004 ... on review by BenBe

   • checked BenBe

   • work done by NEO, pushed to cacert-devel, transfered to testserver
   • needs 2nd review, tested
   • current state:
   • open issues
     1. How are deleted users handled?
     2. Isn't "verified_certs" misleading as the affected tables also contain certs that failed to be signed?
     3. User Statistics don't take removed assurances into account (???)
     4. Why not calculate backwards in the year-dependent loop from the already known values? The loop runs backwards already anyway.
   • the latter is still open

5. bug #1025 Domain Dispute issue

   • BenBe will pickup for 2nd review

   • needs further testing

   • magu, inopiae, u60 -> testing https://bugs.cacert.org/view.php?id=1025

     • several test accounts, variations of one or more email addresses, 0 or 1 domain added
     • test the full disputes procedure for all variations
     • tested by u60

6. 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

7. 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
8. 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

     • OU info from Org cert not stored

     • 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

9. 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

10. 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

    • 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
    • Patch bug #440 was defered (timo's addtl. work), but this project stalls. What to do with bug #440 ?

      • gagern, neo

        bug #440 Problem with subjectAltName

        tested, needs 2nd review, rejected, new deployment getcn/getalt procedure, relates to bug #1101

        {r} 2

      • comments https://bugs.cacert.org/view.php?id=440#c3243, https://bugs.cacert.org/view.php?id=440#c3251 checked?

      • Neo started some fixes (getcn and ...), to be continued
    • ASN.1 parser - planned: incorporate asn.1 from openssl

11. 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

12. New patches

    1. bug #782 Add "notes" field to certificate information

       • inopiae

         bug #782 Add "notes" field to certificate information

         {0}

       • moved to testserver
       • Client certs
       • Current:

         • Renew/Revoke/Delete | Status | Email Address | SerialNumber | Comment | Revoked | Expires | Login

       • move comment to end

         • Renew/Revoke/Delete | Status | Email Address | SerialNumber | Revoked | Expires | Login | Comment | edit

       • create new cert below all mandatory fields?
13. GPG bugs

    1. import prob (eg bug #992 )

       • rfc standards:
         1. rfc 4880, defaults to utf-8, section 5.11

            • https://tools.ietf.org/html/rfc4880

            • RFC 4880 updated by https://tools.ietf.org/html/rfc5581

         2. email rfc-2822

    2. delete/revoke GPG keys (eg bug #1079 )

       • trust signatures can be revoked
       • CRL's have to be added to keyservers, but no one will check
       • revocation: 5 reasons given
       • should be possible, but project needs a developer

14. bug #279 bad domains

    • .*top.*
    • regexp list
    • database table exist
    • update procedure?
    • whats about recuring distribution of update files via cabforum?
    • arbitration?
    • SE console for update?
    • critical admins?
    • check routine on add-domain
    • add domain under OA should be possible ...
    • one-time check of current existing domains ?
      1. first time check against full filter list
      2. individual check in event add domain
      3. global check in event add entry to filter list
      4. replace/update full filter list (case 1 + 4)
    • meta infos:
      1. datasources
      2. attributes (?)
      3. creation date
      4. delete entry / revocation date

15. bug #1121 and bug #1122

    • Assure Someone change request (result from 1122: assure someone, CCA acceptance in Assure someone process)
      1. question: where is the CCA acceptance path of an Assurer?
         • AP defines, Member has accepted CCA by joining CAcert and create an account
         • but there are also "old" assurers who didn't accepted CCA yet
         • 2 known requirements to become an Assurer: 100 APs, CATS passed
         • 3rd yet unverified: CCA acceptance
         • Assurance to be AP conform references to AP
         • Assurance statement gives no statement, that Assurer has accepted CCA
         • AP 4.5 only requests CCA acceptance from Assuree
         • CATS test requires a valid client cert. Client certs can be created without CCA acceptance!!!
         • There is no addtl. CCA acceptance check under CATS
         • so in effect:
           1. an assurer may have created an account before the CCA acceptance request was added to the join form
           2. an assurer may have received upto 100 APs + 50 EPs before CAcert's policy days
           3. giving an assurance doesn't request a CCA acceptance from the Assurer, only the request that assurance is AP compliant

         • so there is no straight verification path that an assurer has accepted CCA except receiving new assurances by -> passive CCA acceptance

      2. assure someone form enhancement
         1. New I verify that the Assuree accepted CCA
         2. Location
         3. Date
         4. assertion
         5. Ap
         6. New I accept CCA
         7. Policies
         8. Points
         9. text (a): I have read and understood the Assurance Policy, the Assurance Handbook and the CCA and am making this Assurance subject to and in compliance with the policy, handbook and CCA.
         10. text (b): I have read and understood the CCA, the Assurance Policy and the Assurance Handbook. I am making this Assurance subject to and in compliance with the CCA, policy and handbook.

         11. AH AP CCA & & dup dup read understood comply.

         12. text (c): I have read and understood the CCA, the Assurance Policy (AP) and the Assurance Handbook (AH). I am making this Assurance subject to and in compliance with the CCA, AP and AH.
    • see also top 4

16. bug #1135 SE activity audit tables

    • addtl. recording of arbitration numbers to members
    • results in long discussions

      • requirements, thought cases (eg name change request while another arbitration is running (-> uncritical))

      • delete account requests handled under precedent case a20111128.3), one "critical" case (certs misusage) is turned in procedure: arbitrator has to follow "emergency case" procedure and to keep track of open "delete account" cases

      • interferance/interaction of 2 of the 3 powers (executive, judicate) (arbitration has to act as executive to forward all new cases to support team with list of open/running arbitration cases)
    • all ends on (arbitration) "critical" cases

    • "critical" cases will be handled under Arbitration eg. a20111128.3 within reasonable (eg 48 hours) window

    • discussion defered

    • 1135 (BenBe) 2nd review by another SA before moving to testserver

    • Michael to review

      1. bug #1135 Extend database table AdminLog et al

17. bug #1136 SE console, delete all certs of a member (instead of highjack an account)

    • probably 1 requirement: addtl. verification step

18. bug #893 Extend Delete account feature for support

    • inopiae

      bug #893 Delete account rev 3 procedure

      needs testing and 2nd review

      {0}

      • test 893 (delete account) with existing server certs, also gpg certs
      • gpg revocation is currently not avail ...
        • manual procedure: currently we hadn't such a case in manualy procedure
        • proposal: in production: set on hold until gpg key expires

        • for testing: gpg keys not expired -> stop procedure if remaining gpg keys not expired

      • if account is locked -> no special exception

      • org admin flag set -> procedure stop (includes Org certs avail)

19. bug #1137 Record the CCA acception for entering an assurance

    • inopiae

      bug #1137 Record the CCA acception for entering an assurance

      needs testing and 2nd review

      {0}

    • 1137 "Record the CCA acception for entering an assurance" needs review testing

20. bug #740 How to become an assurer is missleading

    • according to bug #671 all text pages should be banned within webdb and redirected to a wiki page

    • available 740 fix isn't in compliance to bug #671

    • contrary discussion

21. bug #1141 If i delete Domains, no Servercerts for this domains are listet, even not the revoked

    • moved to testserver

      NEO

      bug #1141 If i delete Domains, no Servercerts for this domains are listet, even not the revoked

      needs testing

      {0}

    • discusssions: arb case? privacy (eg PP 10.), data retention (-> Australien DPA)

    • Marcus to contact Benedikt

22. bug #1008 View for SE to see if user is Organisation Admin for which Organisation Accounts

23. what to do with bug #1143 Web site doesn't scale vertically

5. 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

6. next meeting

• Tuesday, February 12, 2013 22:00 CET

Minutes

1. Preface
   • Fosdem brainstorming

2. bug #1082 The text on the login form is not shown inside the grey box

   • Hendrik, Inopiae

     Bugs #1082 The text on the login form is not shown inside the grey box

     {0}

   • reviewed by Dirk, ok, needs transfer to production
   • Michael to transfer to production?
3. TLS project
   • BenBe/Wytze talked @ fosdem
   • risks fairly low, awaiting fix

4. bug #1119 scheduled visit, not yet announced

   • next scheduled visit on Thursday
5. GPG bugs
   • OpenGPG parser project, reviewed by Michael last weekend
   • Michael remark: using 3x = (===) instead of 2x = (==)
   • unpack (N) 32bit unsigned may become a problem
   • relates to hardware platforms, signer has been replaced about 2 years ago, but needs to be used on both sides (webserver + signer). Webserver upgrade is WIP
   • in principle ok
6. dirk: add oophaga to upcoming board meeting agenda?
7. DEV on bug 1023/1054 "Thawte Patch"
   • no update
8. CAcert installer project
   • Michael got codesigning and tries to add the cert to crypto stick
   • also related "Black Jack" project
   • profiling of use cases for different cert types (client, server, org client, org server)
9. secure boot project (required steps?)
   • we have
     • risk analyze
     • new roots procedure
   • required steps?
     • subroot under eg. org++
     • cps changes
     • new roots?
     • new signer?
     • indirect crl's
10. java installer (question by Marcus)
11. cont. working code signing cert into crypto token for software assessment releases code signing
    • to build new revision of CAcert installer with all lang translations to sign via key on crypto token

Fixed Action Items since last or within meeting

Action Items New

Action items: Meeting Action Items

• CategorySoftwareAssessment