Skip to content

Update woodpecker with file permissions#37

Merged
martinyde merged 5 commits intodevelopfrom
hotfix/permisisons-handling
Apr 1, 2025
Merged

Update woodpecker with file permissions#37
martinyde merged 5 commits intodevelopfrom
hotfix/permisisons-handling

Conversation

@martinyde
Copy link
Contributor

@martinyde martinyde commented Mar 31, 2025

Link to ticket

https://leantime.itkdev.dk/404/?fromDate=2020-01-01&toDate=2050-01-01&userIds=11#/tickets/showTicket/4188

Description

  • Update woodpecker to handle file permisisons hardening.
  • Add drupal setting for skipping permission hardening.

Screenshot of the result

Before:
Skærmbillede 2025-04-01 kl  08 42 28

After:
?
Time will tell....

Checklist

  • My code is covered by test cases.
  • My code passes our test (all our tests).
  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

This PR is not covered by test cases. That is because:

  • It's not within the scope of assignment provided by the PM
  • It's part of a security hotfix and not a new feature
  • It's 3 lines of configuration, two of which i have been instructed to add by a devobs to fix a non-working woodpecker configuration.

Additional comments or questions

Are we there yet?

@martinyde martinyde requested a review from cableman March 31, 2025 12:03
@cableman
Copy link
Contributor

Maybe the commit could have be more informative: "Added chmod commands to woodpecker stg deployment"

@martinyde martinyde changed the title Updated woodpecer Added chmod commands to woodpecker stg deployment Apr 1, 2025
@martinyde martinyde changed the title Added chmod commands to woodpecker stg deployment Update woodpecker with file permissions Apr 1, 2025
@martinyde martinyde requested a review from cableman April 1, 2025 06:55
@martinyde martinyde merged commit 7fd9c22 into develop Apr 1, 2025
5 checks passed
@martinyde martinyde deleted the hotfix/permisisons-handling branch April 1, 2025 08:05
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.

2 participants