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.

Sign up for GitHub

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

Jump to bottom

Fix G302 errors in gosec evaluation #705

Merged
merged 1 commit into from
Mar 26, 2020
Merged

Fix G302 errors in gosec evaluation #705

merged 1 commit into from
Mar 26, 2020

Conversation

bogdandrutu
Copy link
Member

Fix this issue:

[/Users/lazy/github/opentelemetry-collector/exporter/fileexporter/factory.go:70] - G302 (CWE-276): Expect file permissions to be 0600 or less (Confidence: HIGH, Severity: MEDIUM)
  > os.OpenFile(cfg.Path, os.O_RDWR|os.O_CREATE|os.O_TRUNC, 0755)

Updates #672

@bogdandrutu
Fix G302 errors in gosec evaluation
f8416e2 
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
@pjanotti pjanotti merged commit 49a10c2 into open-telemetry:master Mar 26, 2020
@bogdandrutu bogdandrutu deleted the gosec2 branch March 26, 2020 16:39
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
@krnowak
Run generators in all submodules (open-telemetry#705)
2aa6e9f
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
@dependabot
Bump docker in /internal/buildscripts/packaging/release (open-telemet…
ced03e8 
…ry#705)

Bumps [docker](https://github.com/docker/docker-py) from 5.0.0 to 5.0.2.
- [Release notes](https://github.com/docker/docker-py/releases)
- [Commits](docker/docker-py@5.0.0...5.0.2)

---
updated-dependencies:
- dependency-name: docker
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
swiatekm pushed a commit to swiatekm/opentelemetry-collector that referenced this pull request Oct 9, 2024
@puckpuck
[demo] remove observability config section (open-telemetry#705)
185ac3a 
This is a breaking change for anyone that disabled one of the four sub-charts before. UPGRADING.md is updated accordingly.

---------

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pjanotti pjanotti pjanotti approved these changes

@flands flands Awaiting requested review from flands

@owais owais Awaiting requested review from owais

@rghetia rghetia Awaiting requested review from rghetia

@songy23 songy23 Awaiting requested review from songy23

@tigrannajaryan tigrannajaryan Awaiting requested review from tigrannajaryan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bogdandrutu @pjanotti