Skip to content

Skip TLS check and document docker environment variables#19

Merged
arietimmerman merged 4 commits into
limosa-io:mainfrom
Tlepel:skip-tls-check
Oct 9, 2025
Merged

Skip TLS check and document docker environment variables#19
arietimmerman merged 4 commits into
limosa-io:mainfrom
Tlepel:skip-tls-check

Conversation

@Tlepel
Copy link
Copy Markdown
Contributor

@Tlepel Tlepel commented Oct 3, 2025

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for skipping TLS verification when testing SCIM servers with self-signed certificates and improves documentation of Docker environment variables.

  • Adds skipTlsVerification configuration option to bypass SSL certificate validation
  • Documents all environment variables in Docker documentation with a proper table
  • Updates code formatting from ~~~ to standard ```` markdown blocks

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/helpers.js Adds HTTPS agent configuration to skip TLS verification when enabled
site/docker/index.md Documents environment variables and updates markdown formatting
server.js Sets default value for SKIP_TLS_VERIFICATION environment variable
scim.test.js Adds skipTlsVerification configuration from environment variable
README.md Documents new --skip-tls-verification command line option

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread site/docker/index.md Outdated
Comment thread server.js Outdated
arietimmerman and others added 2 commits October 9, 2025 22:21
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@arietimmerman arietimmerman merged commit 0dd939a into limosa-io:main Oct 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants