Test before merge #2

Merged
Draak merged 18 commits from testing into master 2019-01-12 23:02:23 +00:00
Owner

test befor Merge testing into master

test befor Merge testing into master
Draak self-assigned this 2019-01-12 19:40:06 +00:00
README.md Outdated
@ -24,2 +22,3 @@
Here you will find the files and commands used to create your pki.
Un petit point sur le contenue de ce git
Let's take stock of the content of this git:
Owner

Let's take a look at me semble plus utilisé en anglais courant

*Let's take a look at* me semble plus utilisé en anglais courant
Author
Owner

Done.

Done.
README.md Outdated
@ -27,2 +26,2 @@
Deux dossiers, "conf" et "script".
Conf contient les fichiers de configuration des differents types de certificats et script de quoi generer les différents types de certificats.
Two folders, "conf" and "script".
Conf contains the configuration files of the different types of certificates and script of what to generate the different types of certificates.
Owner

Utiliser le markdown pour bien détacher les noms de fichiers/dossiers du reste du texte: conf, script

`conf`
`script`
Utiliser le markdown pour bien détacher les noms de fichiers/dossiers du reste du texte: `conf`, `script` ``` `conf` `script` ```
Author
Owner

Done.

Done.
README.md Outdated
@ -31,2 +29,3 @@
At the root you will find setup.sh which will create the directories and files necessary to generate the PKI. But also to generate the first four certificates authority, root, network, identity and component.
Parlons en justement voici à quoi va ressembler notre chaine
Let's talk about exactly what our channel will look like
Owner

channel => chain

channel => chain
Author
Owner

Done.

Done.
README.md Outdated
@ -171,1 +168,3 @@
Les premiers CRL sont également créer, mais il faudrat pas oublier de le générer si vous en avez besoin.
So you have just generated your certificate chain with the keys and CSR that go with it.
But also the concatenations of certificates for the files chains.
The first CRLs are also created, but it will be necessary not to forget to generate it if you need it.
Owner

J'ai pas compris la nuance, si les CRL sont créées au lancement du script, pourquoi devoir les générer ?

J'ai pas compris la nuance, si les CRL sont créées au lancement du script, pourquoi devoir les générer ?
Author
Owner

en fait on les RE génere, j'ai modifié pour la précision

en fait on les RE génere, j'ai modifié pour la précision
README.md Outdated
@ -176,3 +175,3 @@
<a name="server"></a>
### 4.1 - TLS Server
On va utiliser le script dédié à savoir conf/new_server_cert.sh. Il doit être suivi d'un ou plusieurs arguments. A savoir des TLD Valide. Exemple de commande:
We will use the dedicated script to know conf/new_server_cert.sh. It must be followed by one or more arguments. Namely valid TLDs. Sample order:
Owner

"We will use the dedicated script named conf/new_server_cert.sh. It must be followed by one or more arguments, namely valid TLDs"

"We will use the dedicated script named `conf/new_server_cert.sh`. It must be followed by one or more arguments, namely valid TLDs"
Author
Owner

Done.

Done.
README.md Outdated
@ -187,3 +186,3 @@
<a name="client"></a>
### 4.2 - TLS Client
Encore une fois, il y a un script dédié à la création, conf/new_client_cert.sh. Il doit être suivi de deux arguments, le prénom suivi du nom de l'utilisateur. Exemple de commande:
Again, there is a script dedicated to the creation, conf/new_client_cert.sh. It must be followed by two arguments, the first name followed by the user's name. Sample order:
Owner

... dedicated to the client certificate creation...

... dedicated to the **client certificate** creation...
Author
Owner

Done.

Done.
README.md Outdated
@ -195,3 +194,3 @@
<a name="ident"></a>
### 4.3 - Identity
Devinez? Il y a un script dédié à la création, conf/new_ident_cert.sh. Il doit être suivi de deux arguments, le prénom suivi du nom de l'utilisateur. Exemple de commande:
Guess? There is a script dedicated to the creation, conf/new_ident_cert.sh. It must be followed by three arguments, the first followed by the name of the user and the email address. Sample order:
Owner

Guess what?

Guess **what**?
Author
Owner

Done.

Done.
README.md Outdated
@ -202,3 +200,4 @@
This will generate a key, a CSR, a certificate and a pkcs12 bundle named "mime-ident-Walter-KOVACS.p12"
<a name="encript"></a>
### 4.4 - Encription
Owner

Encryption

Encr**y**ption
Author
Owner

Done.

Done.
@ -7,3 +7,3 @@
aia_url = $base_url/$ca.cer # CA certificate URL
crl_url = $base_url/$ca.crl # CRL distribution point
ocsp_url = http://ocsp.barbed.fr # OCSP responder URL
ocsp_url = http://ocsp.SCRIPT_ROOTDOMAIN # OCSP responder URL
Owner

ça devrait être https plutôt que http?

ça devrait être https plutôt que http?
Author
Owner

Done.

Done.
@ -26,2 +20,3 @@
commonName = "6. Common Name (eg, Walter KOVACS)"
commonName_max = 64
emailAddress = "7. Email Address (eg, yduval@barbed.fr)"
emailAddress = "7. Email Address (eg, wkovacs@Watchmen.fr)"
Owner

watchmen.io ? ;)

watchmen.io ? ;)
Author
Owner

Done.

Done.
@ -26,2 +20,3 @@
commonName = "6. Common Name (eg, Walter KOVACS)"
commonName_max = 64
emailAddress = "7. Email Address (eg, yduval@barbed.fr)"
emailAddress = "7. Email Address (eg, wkovacs@Watchmen.fr)"
Owner

watchmen.io aussi ?

watchmen.io aussi ?
Author
Owner

Done.

Done.
@ -0,0 +1,9 @@
#!/bin/sh
Owner

Ajouter un commentaire sur ce qu'on obtient à la fin, à savoir une CRL pour component-ca et identity-ca chacun, mais pas pour les CA en amont.

Ajouter un commentaire sur ce qu'on obtient à la fin, à savoir une CRL pour `component-ca` et `identity-ca` chacun, mais pas pour les CA en amont.
Author
Owner

You will create two CRL, first one for component-ca et the other one for identity-ca

Done.

# You will create two CRL, first one for component-ca et the other one for identity-ca Done.
@ -40,0 +43,4 @@
read -p "2. State or Province Name (eg, state) : " stateOrProvinceName
read -p "3. Locality Name (eg, city) : " localityName
read -p "4. Organization Name (eg, Watchmen) : " organizationName
read -p "5. Root Domain (eg, watchmen.com) : " rootdomain
Owner

On reste cohérent: watchmen.io :p

On reste cohérent: watchmen.io :p
Author
Owner

Done.

Done.
setup.sh Outdated
@ -40,0 +68,4 @@
sed -i -e "s/SCRIPT_COUNTRY/$countryName/g" -e "s/SCRIPT_ORGANIZATIONALNAME/$organizationName/g" -e "s/SCRIPT_ROOTDOMAIN/$rootdomain/g" conf/component-ca.conf
sed -i -e "s/SCRIPT_COUNTRY/$countryName/g" -e "s/SCRIPT_ORGANIZATIONALNAME/$organizationName/g" -e "s/SCRIPT_ROOTDOMAIN/$rootdomain/g" conf/identity-ca.conf
# Modify Script files
Owner

Idée d'amélioration: au lieu de hardcoder les valeurs dans les scripts, créer un script de variables qui se fait substituer avec sed comme ici, mais les scripts l'importent en "source".

Un peu de doc: https://stackoverflow.com/questions/16011245/source-files-in-a-bash-script

Idée d'amélioration: au lieu de hardcoder les valeurs dans les scripts, créer un script de variables qui se fait substituer avec `sed` comme ici, mais les scripts l'importent en "source". Un peu de doc: https://stackoverflow.com/questions/16011245/source-files-in-a-bash-script
setup.sh Outdated
@ -49,3 +95,3 @@
# Création des bases de donnée
# Creating database
cp /dev/null ca/root-ca/db/root-ca.db
Owner

touch plutôt ? :p

`touch` plutôt ? :p
Author
Owner

Done.

Done.
setup.sh Outdated
@ -79,3 +125,3 @@
-out ca/root-ca.crt \
-extensions root_ca_ext \
-enddate 20391231235959Z
-enddate 20391231235959Z \
Owner

hardcodé ? Il est possible de calculer une durée de 10 ans à partir de la date courante.

hardcodé ? Il est possible de calculer une durée de 10 ans à partir de la date courante.
Author
Owner

Added
Lifetime of root-ca and network-ca, today + 10 years
LIFETIME="$(date +%Y%m%d%H%M%S -d '+10 years')"

Added Lifetime of root-ca and network-ca, today + 10 years LIFETIME="$(date +%Y%m%d%H%M%S -d '+10 years')"
setup.sh Outdated
@ -101,3 +148,3 @@
-out ca/network-ca.crt \
-extensions intermediate_ca_ext \
-enddate 20391231235959Z
-enddate 20391231235959Z \
Owner

hardcodé encore

hardcodé encore
Author
Owner

Added
Lifetime of root-ca and network-ca, today + 10 years
LIFETIME="$(date +%Y%m%d%H%M%S -d '+10 years')"

Added Lifetime of root-ca and network-ca, today + 10 years LIFETIME="$(date +%Y%m%d%H%M%S -d '+10 years')"
Draak closed this pull request 2019-01-12 23:02:23 +00:00
Draak deleted branch testing 2019-01-12 23:03:32 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
barbedfr/pki!2
No description provided.