Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

[R4R] change file permission and add error handling #167

Merged
merged 2 commits into from
Apr 20, 2022
Merged

[R4R] change file permission and add error handling #167

merged 2 commits into from
Apr 20, 2022

Conversation

forcodedancing
Copy link
Contributor

@forcodedancing forcodedancing commented Apr 14, 2022 •
edited
Loading

Description

This pr fixes some un-checked errors and corrects the permissions of created files.

Rationale

  • handle errors when closing files for tool json2wal
  • change file permission to 0600

Example

n/a

Changes

  • changed file permission
  • add error handling

@forcodedancing forcodedancing changed the base branch from pre-opensource to develop April 14, 2022 13:37
@forcodedancing forcodedancing merged commit c934a76 into develop Apr 20, 2022
@forcodedancing forcodedancing mentioned this pull request Apr 26, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@qinglin89 qinglin89 qinglin89 approved these changes

@unclezoro unclezoro unclezoro approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@forcodedancing @unclezoro @qinglin89