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.

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

TODOs #1

Open
1 of 9 tasks
hakimifr opened this issue Nov 6, 2024 · 3 comments
Open
1 of 9 tasks

TODOs #1

hakimifr opened this issue Nov 6, 2024 · 3 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed
Milestone

Comments

@hakimifr
Copy link
Contributor

hakimifr commented Nov 6, 2024

ksau-go TODOs checklist

Note: this list is based off the help message, including some extra details.

  • Implement -r, -q, -c flags
    • -r flag (Add random string to file name)
    • -q flag (suppress all output except file link)
    • -c flag1 (flag to choose any remote, overriding auto behavior)
  • Implement list command2
    • does it list the available remotes?
    • does it show total, used, free, trashed?
  • Implement update command
    Self explanatory, update ksau in-place
  • Implement version command
    Also self explanatory, show current ksau's version

Footnotes

  1. This flag can only be implemented once we switch the upload behavior to behave like old ksau. Where for files <100MB, a random remote is chosen. For files >=100MB, remote with the most free space is chosen. Currently ksau upload's argument is <remote> <file> <folder>, and we want to get rid of <remote>. The old implementation also caches the remote with most free space to avoid spamming same request to MS's API.

  2. Even better if we can add flags to customize the output.

@hakimifr hakimifr added enhancement New feature or request help wanted Extra attention is needed labels Nov 6, 2024
@global-index-source global-index-source deleted a comment from hakimifr Nov 6, 2024
@hakimifr
Copy link
Contributor Author

hakimifr commented Nov 6, 2024

@hakimifr @ksauraj see if you need to add anything

@ksauraj
Copy link
Member

ksauraj commented Nov 6, 2024

I will add things as we make progress through different implementation.

@ksauraj ksauraj added this to the v2.0.0-alpha milestone Nov 6, 2024
@ksauraj
Copy link
Member

ksauraj commented Nov 6, 2024

image

  • It seems we are unable to fetch the ClientId and ClientSecret form the decrypted configfile .
  • Severity: Unknown ( as we are not using it anywhere for now )
  • Cause: The decrypted json string does not have both the keys
  • Fix: Update the current config with required values or remove both the vars

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants