Skip to content

Commit 0e7a7b4

Browse files
kdwilskdwils
committed
pr review fixes
1 parent 37f0b57 commit 0e7a7b4

10 files changed

Lines changed: 300 additions & 346 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: Lint and Test Charts
33
on:
44
push:
55
branches:
6-
- '**'
6+
- "**"
77

88
jobs:
99
lint-test:
@@ -21,7 +21,7 @@ jobs:
2121

2222
- uses: actions/setup-python@v5.3.0
2323
with:
24-
python-version: '3.x'
24+
python-version: "3.x"
2525
check-latest: true
2626

2727
- name: Set up chart-testing
@@ -45,4 +45,4 @@ jobs:
4545

4646
- name: Run chart-testing (install)
4747
if: steps.list-changed.outputs.changed == 'true'
48-
run: ct install --config=ct.yaml --target-branch ${{ github.event.repository.default_branch }}
48+
run: ct install --config=ct.yaml --target-branch ${{ github.event.repository.default_branch }}

‎.github/workflows/release.yml‎

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: Release Charts
33
on:
44
push:
55
tags:
6-
- '*'
6+
- "*"
77

88
jobs:
99
release:
@@ -16,12 +16,7 @@ jobs:
1616
with:
1717
fetch-depth: 0
1818

19-
- name: Configure Git
20-
run: |
21-
git config user.name "$GITHUB_ACTOR"
22-
git config user.email "$GITHUB_ACTOR@users.noreply.github.com"
23-
2419
- name: Run chart-releaser
2520
uses: helm/chart-releaser-action@v1.7.0
2621
env:
27-
CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
22+
CR_TOKEN: "${{ secrets.GITHUB_TOKEN }}"

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Tinyauth helm
22

3-
A helm chart for running Tinyauth in a kubernetes environment.
3+
A helm chart for running Tinyauth in a Kubernetes cluster.
44

55
## Development
66

‎charts/tinyauth/Chart.yaml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
apiVersion: v2
22
name: tinyauth
3-
description: A Helm chart for Kubernetes
3+
description: The simplest way to protect your apps with a login screen.
44

55
# A chart can be either an 'application' or a 'library' chart.
66
#
@@ -21,4 +21,4 @@ version: 0.1.0
2121
# incremented each time you make changes to the application. Versions are not expected to
2222
# follow Semantic Versioning. They should reflect the version the application is using.
2323
# It is recommended to use it with quotes.
24-
appVersion: "v3.6.2"
24+
appVersion: "v4.0.0"

‎charts/tinyauth/README.md‎

Lines changed: 27 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
# tinyauth
22

3-
![Version: 0.1.0](https://img.shields.io/badge/Version-0.1.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v3.6.2](https://img.shields.io/badge/AppVersion-v3.6.2-informational?style=flat-square)
3+
![Version: 0.1.0](https://img.shields.io/badge/Version-0.1.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v4.0.0](https://img.shields.io/badge/AppVersion-v4.0.0-informational?style=flat-square)
44

5-
A Helm chart for Kubernetes
5+
The simplest way to protect your apps with a login screen.
66

77
## Values
88

@@ -33,14 +33,20 @@ A Helm chart for Kubernetes
3333
| ingress.hosts[0].paths[0].path | string | `"/"` | |
3434
| ingress.hosts[0].paths[0].pathType | string | `"ImplementationSpecific"` | |
3535
| ingress.tls | list | `[]` | |
36-
| livenessProbe.httpGet.path | string | `"/api/healthcheck"` | |
36+
| livenessProbe.httpGet.path | string | `"/api/health"` | |
3737
| livenessProbe.httpGet.port | string | `"http"` | |
3838
| nameOverride | string | `""` | |
3939
| nodeSelector | object | `{}` | |
40+
| persistence.accessMode | string | `"ReadWriteOnce"` | |
41+
| persistence.annotations | object | `{}` | |
42+
| persistence.enabled | bool | `false` | |
43+
| persistence.mountPath | string | `"/data"` | |
44+
| persistence.size | string | `"1Gi"` | |
45+
| persistence.storageClass | string | `""` | |
4046
| podAnnotations | object | `{}` | |
4147
| podLabels | object | `{}` | |
4248
| podSecurityContext | object | `{}` | |
43-
| readinessProbe.httpGet.path | string | `"/api/healthcheck"` | |
49+
| readinessProbe.httpGet.path | string | `"/api/health"` | |
4450
| readinessProbe.httpGet.port | string | `"http"` | |
4551
| replicaCount | int | `1` | |
4652
| resources | object | `{}` | |
@@ -55,52 +61,42 @@ A Helm chart for Kubernetes
5561
| tinyauth.general.appTitle | string | `""` | |
5662
| tinyauth.general.appUrl | string | `""` | |
5763
| tinyauth.general.backgroundImage | string | `""` | |
58-
| tinyauth.general.cookieSecure | bool | `false` | |
5964
| tinyauth.general.disableContinue | bool | `false` | |
6065
| tinyauth.general.forgotPasswordMessage | string | `""` | |
61-
| tinyauth.general.logLevel | int | `1` | |
66+
| tinyauth.general.logLevel | string | `"info"` | |
6267
| tinyauth.general.loginMaxRetries | int | `5` | |
6368
| tinyauth.general.loginTimeout | int | `300` | |
6469
| tinyauth.general.oauthAutoRedirect | string | `""` | |
6570
| tinyauth.general.oauthWhitelist | string | `""` | |
6671
| tinyauth.general.port | int | `3000` | |
67-
| tinyauth.general.secret | string | `""` | |
68-
| tinyauth.general.secretFile | string | `""` | |
69-
| tinyauth.general.secretSecretRef.key | string | `""` | |
70-
| tinyauth.general.secretSecretRef.name | string | `""` | |
72+
| tinyauth.general.secureCookie | bool | `false` | |
7173
| tinyauth.general.sessionExpiry | int | `86400` | |
7274
| tinyauth.general.users | string | `""` | |
7375
| tinyauth.general.usersFile | string | `""` | |
74-
| tinyauth.generic.authUrl | string | `""` | |
75-
| tinyauth.generic.clientId | string | `""` | |
76-
| tinyauth.generic.clientSecret | string | `""` | |
77-
| tinyauth.generic.clientSecretFile | string | `""` | |
78-
| tinyauth.generic.clientSecretSecretRef.key | string | `""` | |
79-
| tinyauth.generic.clientSecretSecretRef.name | string | `""` | |
80-
| tinyauth.generic.name | string | `""` | |
81-
| tinyauth.generic.scopes | string | `""` | |
82-
| tinyauth.generic.skipSsl | bool | `false` | |
83-
| tinyauth.generic.tokenUrl | string | `""` | |
84-
| tinyauth.generic.userUrl | string | `""` | |
85-
| tinyauth.github.clientId | string | `""` | |
86-
| tinyauth.github.clientSecret | string | `""` | |
87-
| tinyauth.github.clientSecretFile | string | `""` | |
88-
| tinyauth.github.clientSecretSecretRef.key | string | `""` | |
89-
| tinyauth.github.clientSecretSecretRef.name | string | `""` | |
90-
| tinyauth.google.clientId | string | `""` | |
91-
| tinyauth.google.clientSecret | string | `""` | |
92-
| tinyauth.google.clientSecretFile | string | `""` | |
93-
| tinyauth.google.clientSecretSecretRef.key | string | `""` | |
94-
| tinyauth.google.clientSecretSecretRef.name | string | `""` | |
9576
| tinyauth.ldap.address | string | `""` | |
9677
| tinyauth.ldap.baseDn | string | `""` | |
9778
| tinyauth.ldap.bindDn | string | `""` | |
9879
| tinyauth.ldap.bindPassword | string | `""` | |
9980
| tinyauth.ldap.bindPasswordSecretRef.key | string | `""` | |
10081
| tinyauth.ldap.bindPasswordSecretRef.name | string | `""` | |
82+
| tinyauth.ldap.enabled | bool | `false` | |
10183
| tinyauth.ldap.insecure | bool | `false` | |
10284
| tinyauth.ldap.searchFilter | string | `"(uid=%s)"` | |
85+
| tinyauth.providers[0].authUrl | string | `""` | |
86+
| tinyauth.providers[0].clientId | string | `""` | |
87+
| tinyauth.providers[0].clientSecret | string | `""` | |
88+
| tinyauth.providers[0].clientSecretFile | string | `""` | |
89+
| tinyauth.providers[0].clientSecretSecretRef.key | string | `""` | |
90+
| tinyauth.providers[0].clientSecretSecretRef.name | string | `""` | |
91+
| tinyauth.providers[0].id | string | `""` | |
92+
| tinyauth.providers[0].name | string | `""` | |
93+
| tinyauth.providers[0].scopes | string | `""` | |
94+
| tinyauth.providers[0].skipSsl | bool | `false` | |
95+
| tinyauth.providers[0].tokenUrl | string | `""` | |
96+
| tinyauth.providers[0].userInfoUrl | string | `""` | |
10397
| tolerations | list | `[]` | |
10498
| volumeMounts | list | `[]` | |
10599
| volumes | list | `[]` | |
106100

101+
----------------------------------------------
102+
Autogenerated from chart metadata using [helm-docs v1.14.2](https://github.com/norwoodj/helm-docs/releases/v1.14.2)

‎charts/tinyauth/ci/all-features-values.yaml‎

Lines changed: 27 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -40,39 +40,38 @@ tinyauth:
4040
port: 3000
4141
address: "0.0.0.0"
4242
appUrl: "http://tinyauth-test.local"
43-
secret: "a7b9c2d4e6f8g1h3i5j7k9l2m4n6o8p1"
4443
users: "user:$$2a$$10$$UdLYoJ5lgPsC0RKqYH/jMua7zIn0g9kPqWmhYayJYLaZQ/FTmH2/u"
45-
cookieSecure: false
44+
secureCookie: false
4645
disableContinue: false
4746
oauthWhitelist: "*"
4847
sessionExpiry: 3600
49-
logLevel: 1
48+
logLevel: info
5049
appTitle: "TinyAuth Test"
5150
loginMaxRetries: 3
5251
loginTimeout: 300
5352
forgotPasswordMessage: "Contact admin for password reset"
5453
oauthAutoRedirect: "github"
5554
backgroundImage: "/static/bg.jpg"
5655

57-
github:
58-
clientId: "test-github-client-id"
59-
clientSecret: "test-github-client-secret"
60-
61-
google:
62-
clientId: "test-google-client-id"
63-
clientSecret: "test-google-client-secret"
64-
65-
generic:
66-
clientId: "test-generic-client-id"
67-
clientSecret: "test-generic-client-secret"
68-
authUrl: "https://example.com/oauth/authorize"
69-
tokenUrl: "https://example.com/oauth/token"
70-
userUrl: "https://example.com/oauth/user"
71-
scopes: "openid profile email"
72-
name: "Generic OAuth"
73-
skipSsl: false
56+
providers:
57+
- id: github
58+
clientId: "test-github-client-id"
59+
clientSecret: "test-github-client-secret"
60+
- id: google
61+
clientId: "test-google-client-id"
62+
clientSecret: "test-google-client-secret"
63+
- id: custom
64+
clientId: "test-custom-client-id"
65+
clientSecret: "test-custom-client-secret"
66+
authUrl: "https://example.com/oauth/authorize"
67+
tokenUrl: "https://example.com/oauth/token"
68+
userInfoUrl: "https://example.com/oauth/user"
69+
scopes: "openid profile email"
70+
name: "Custom OAuth"
71+
skipSsl: false
7472

7573
ldap:
74+
enabled: true
7675
address: "ldap://test-ldap:389"
7776
bindDn: "cn=admin,dc=example,dc=com"
7877
bindPassword: "test-ldap-password"
@@ -94,14 +93,14 @@ ingress:
9493

9594
livenessProbe:
9695
httpGet:
97-
path: /
96+
path: /api/health
9897
port: http
9998
initialDelaySeconds: 30
10099
periodSeconds: 10
101100

102101
readinessProbe:
103102
httpGet:
104-
path: /
103+
path: /api/health
105104
port: http
106105
initialDelaySeconds: 5
107106
periodSeconds: 5
@@ -112,3 +111,9 @@ autoscaling:
112111
maxReplicas: 1
113112
targetCPUUtilizationPercentage: 70
114113
targetMemoryUtilizationPercentage: 80
114+
115+
httpRoute:
116+
enabled: false
117+
118+
persistence:
119+
enabled: false

0 commit comments

Comments
 (0)