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

Onedrive_Backup #1081

Merged
merged 30 commits into from
Oct 17, 2023
Merged

Onedrive_Backup #1081

merged 30 commits into from
Oct 17, 2023

Conversation

MartinRinas
Copy link
Contributor

fixed cryptography dependency for ARM architecture

@MartinRinas
initial version
67c74af
@MartinRinas
update requirements.txt
5114adc
@MartinRinas
draft 2
115517b
@MartinRinas
retrieve scope from config
085024d
@MartinRinas
prepare config
005199a
@MartinRinas
save persistent token cahe
616309d
@MartinRinas
save token cache
1994012
@MartinRinas
auth flow, token cache
605e975
@MartinRinas
minor flake8 fixes
4d000ba
@MartinRinas
clean up comments
9f8740f
@MartinRinas
flake8, refactor
4d70337
@MartinRinas
Update publish_docs_to_wiki.yml
19a9835
@MartinRinas
move logic into api.py
26d65c4
@MartinRinas
formatting
ca20778
@MartinRinas
Merge branch 'master' of https://github.com/MartinRinas/core into One…
8a0d9b9 
…drive_Backup
@MartinRinas
Revert "Update publish_docs_to_wiki.yml"
23a2d1f 
This reverts commit 19a9835.
@MartinRinas
flake8
d93a824
@MartinRinas
add required module to workflow
f225b95
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core into Onedrive…
4287788 
…_Backup
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core into Onedrive…
afbeef5 
…_Backup
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core into Onedrive…
a17f908 
…_Backup
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core into Onedrive…
c73bd60 
…_Backup
@MartinRinas
remove sharepoint-onedrive SDK
4bcbfc1
@MartinRinas
add cryptograhpy
124f1d2
@MartinRinas
add cryptography==40.0.1
392b55b
@benderl
Copy link
Contributor

benderl commented Aug 14, 2023

Bitte nochmal auf den aktuellen Master bringen, um die Anpassungen von #1085 zu prüfen.

@benderl benderl added this to the 2.2.0 milestone Aug 14, 2023
@benderl benderl added the enhancement New feature or request label Aug 14, 2023
@MartinRinas
Copy link
Contributor Author

MartinRinas commented Oct 16, 2023 •
edited
Loading

@LKuemmel ich dachte mir ich bereite den PR mal vor und ziehe alle angefallenen Änderungen nach, ist ja doch einige Zeit vergangen. Leider schlägt das Test-Discovery fehl, ich kann leider nicht erkennen woran das nun liegen könnte?

Das Modul an sich funktioniert problemlos, ich kann mich an OneDrive anmelden, Tokens erzeugen und eine Sicherung erstellen. Rein funktional tut also alles, das MSAL Modul lässt sich importieren etc. Warum das mit den Tests nicht klappt verstehe ich aber leider nicht.

image

image

@MartinRinas
Copy link
Contributor Author

Die Änderungen in der packages/modules/conftest.py in diesem PR scheinen das Test Discovery in diesem PR aus der Bahn zu werfen.

#1115

Wenn ich hier Z18, 34, 35, und 36 auskommentiere läuft das wieder durch.

image

@LKuemmel
Copy link
Contributor

LKuemmel commented Oct 17, 2023 •
edited
Loading

image

Mit einem Mock für das msal-Modul läuft es durch.

@LKuemmel
Copy link
Contributor

Die anderen Änderungen aus dem Diff müsstest Du auch bitte noch umsetzen, damit es durch läuft.

@LKuemmel LKuemmel merged commit 0fc763a into openWB:master Oct 17, 2023
1 check passed
@MartinRinas MartinRinas deleted the Onedrive_Backup branch October 17, 2023 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LKuemmel LKuemmel LKuemmel approved these changes

Assignees

@LKuemmel LKuemmel

Labels
enhancement New feature or request
Projects
None yet
Milestone
2.1.2
Development

Successfully merging this pull request may close these issues.
3 participants
@MartinRinas @benderl @LKuemmel