Skip to content

feat(core): fix token leak in OneDrive#3470

Merged
Xuanwo merged 3 commits intoapache:mainfrom
morristai:feat/fix_some_bugs
Nov 3, 2023
Merged

feat(core): fix token leak in OneDrive#3470
Xuanwo merged 3 commits intoapache:mainfrom
morristai:feat/fix_some_bugs

Conversation

@morristai
Copy link
Copy Markdown
Member

@morristai morristai commented Nov 3, 2023

Description

  1. Hide token info in OneDrive
  2. Format import sequence and path
  3. Drop async as there's no await inside

@morristai morristai requested a review from Xuanwo as a code owner November 3, 2023 02:28
@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Nov 3, 2023
@Xuanwo Xuanwo changed the title feat(core): fix some bugs feat(core): fix token leak in OneDrive Nov 3, 2023
Xuanwo
Xuanwo approved these changes Nov 3, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit b5a29b9 into apache:main Nov 3, 2023
@silver-ymz silver-ymz mentioned this pull request Nov 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Xuanwo Xuanwo Xuanwo approved these changes

Assignees

No one assigned

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@morristai @Xuanwo