Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: #161, #301, #291, #198, #143, #121, #119, #210, #246, #214, #141, #144, #212, #316 #337

Merged
merged 37 commits into from
Feb 28, 2024

Conversation

MSzalowski
Copy link
Contributor

@MSzalowski MSzalowski commented Feb 28, 2024

Sworzen1 and others added 30 commits February 23, 2024 14:47
…ment

This commit updates the user_data.sh script within the Terraform modules
for govtool-EC2 instances. It adds Michał Szałowski to the list of users
authorized for SSH access, thereby expanding the secure access
management system to include a team leader. This change is a step
towards fulfilling the secure credential sharing acceptance criterion by
ensuring new team members like Michał have the necessary access to
sensitive environments.
Extend the drep/info endpoint by adding isRegisteredAsSoleVoter and wasRegisteredAsSoleVoter fields
jankun4 and others added 7 commits February 27, 2024 11:24
Signed-off-by: jankun4 <michaljankun@gmail.com>
Signed-off-by: jankun4 <michaljankun@gmail.com>
Updated CSP settings in Traefik configuration for beta, dev, staging,
and test environments to include Sentry's domain without the ingest
subdomain. This change permits Sentry error logging and reporting to
function correctly without being blocked by the CSP. The adjustment
ensures Sentry can capture and report runtime errors, facilitating
better monitoring and debugging capabilities across our development,
testing, and staging phases.

- Modified CSP `connect-src` directive to set
  `o4506155985141760.ingest.sentry.io` replacing old Sentry
  sources.
- Ensured that the updated settings adhere to our security policies by
  only allowing necessary and trusted sources.

This update addresses the need for comprehensive error reporting through
Sentry, enhancing our ability to quickly identify and resolve issues in
our application's environments.
- Integrated OAuth for secure Slack deployment notifications #194.
- Streamlined build and deployment process for faster CD and reduced
  resource use #246.
- Allow Sentry error reporting in CSP settings across environments #291.
This commit introduces a significant update to the `.env.example` file
within the `scripts/govtool` directory to enhance usability and prevent
configuration mistakes. Key changes include:

- Addition of Vim modeline at the beginning of the file (`# vim: set
  ft=bash`) to ensure that when editing the file with Vim, the correct
  filetype (bash) is automatically set. This facilitates syntax
  highlighting and other filetype-specific features, improving the editing
  experience.
- Introduction of a conditional warning mechanism that activates when
  the environment is not set to 'dev'. This mechanism employs bash
  scripting to alert to users, especially those working in
  non-development environments, to proceed with caution and be aware of
  their current configuration context. The warning message explicitly
  states the current environment value, reinforcing the need for
  attentiveness.

These enhancements are designed to minimize the risk of misconfiguration
by providing clear, environment-specific warnings and improving file
readability and editing support.
Signed-off-by: jankun4 <michaljankun@gmail.com>
@MSzalowski MSzalowski marked this pull request as draft February 28, 2024 11:04
@MSzalowski MSzalowski changed the title test: #161, #301 test: #161, #301, #291, #198, #143, #121, #119, #210, #246, #214, #141, #144, #212, #316 Feb 28, 2024
@MSzalowski MSzalowski marked this pull request as ready for review February 28, 2024 12:42
@MSzalowski MSzalowski merged commit 21b5a78 into test Feb 28, 2024
5 checks 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.

6 participants