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

Changing cache location and deleting old files #414

Merged
merged 8 commits into from
Nov 3, 2023
Merged

Changing cache location and deleting old files #414

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7a12ef9
Changing cache location and deleting old files
Oct 30, 2023
7a58599
Merge branch 'main' into cache/file-location
alex-fang0 Oct 30, 2023
d93abea
Moving file check to middleware
Oct 31, 2023
a0a9537
Merge branch 'cache/file-location' of github.com:Brightspace/bmx into…
Oct 31, 2023
bf81251
Removing unused code
Oct 31, 2023
b76b527
Catching on file management error
Oct 31, 2023
d306162
Auto format cache/file-location (#415)
github-actions[bot] Oct 31, 2023
97ecd22
Console Error instead
Oct 31, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions src/D2L.Bmx/BmxConfigProvider.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,9 +52,6 @@ public BmxConfig GetConfiguration() {
}

public void SaveConfiguration( BmxConfig config ) {
if( !Directory.Exists( BmxPaths.BMX_DIR ) ) {
Directory.CreateDirectory( BmxPaths.BMX_DIR );
}
var op = new FileStreamOptions {
Mode = FileMode.OpenOrCreate,
Access = FileAccess.ReadWrite,
Expand Down
7 changes: 4 additions & 3 deletions src/D2L.Bmx/BmxPaths.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@ namespace D2L.Bmx;
internal static class BmxPaths {
public static readonly string USER_HOME_DIR = Environment.GetFolderPath( Environment.SpecialFolder.UserProfile );
public static readonly string BMX_DIR = Path.Join( USER_HOME_DIR, ".bmx" );
public static readonly string SESSIONS_FILE_NAME = Path.Join( BMX_DIR, "sessions" );
public static readonly string CACHE_DIR = Path.Join( BMX_DIR, "\\cache" );
public static readonly string SESSIONS_FILE_NAME = Path.Join( CACHE_DIR, "sessions" );
public static readonly string CONFIG_FILE_NAME = Path.Join( BMX_DIR, "config" );
public static readonly string UPDATE_CHECK_FILE_NAME = Path.Join( BMX_DIR, "update_check" );
public static readonly string AWS_CREDS_CACHE_FILE_NAME = Path.Join( BMX_DIR, "awsCredsCache" );
public static readonly string UPDATE_CHECK_FILE_NAME = Path.Join( CACHE_DIR, "updateCheck" );
public static readonly string AWS_CREDS_CACHE_FILE_NAME = Path.Join( CACHE_DIR, "awsCreds" );
}
3 changes: 0 additions & 3 deletions src/D2L.Bmx/Okta/OktaSessionStorage.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,6 @@ internal interface IOktaSessionStorage {

internal class OktaSessionStorage : IOktaSessionStorage {
void IOktaSessionStorage.SaveSessions( List<OktaSessionCache> sessions ) {
if( !Directory.Exists( BmxPaths.BMX_DIR ) ) {
Directory.CreateDirectory( BmxPaths.BMX_DIR );
}

string jsonString = JsonSerializer.Serialize(
sessions,
Expand Down
11 changes: 11 additions & 0 deletions src/D2L.Bmx/Program.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,17 @@
next
) => {
await UpdateChecker.CheckForUpdatesAsync( configProvider.GetConfiguration() );
alex-fang0 marked this conversation as resolved.
Show resolved Hide resolved
//Creating new Cache directory and removing old files.
if( !Directory.Exists( BmxPaths.CACHE_DIR ) ) {
try {
Directory.CreateDirectory( BmxPaths.CACHE_DIR );
File.Delete( Path.Join( BmxPaths.BMX_DIR, "awsCredsCache" ) );
File.Delete( Path.Join( BmxPaths.BMX_DIR, "sessions" ) );
File.Delete( Path.Join( BmxPaths.BMX_DIR, "update_check" ) );
} catch ( Exception ex ) {

Check warning on line 225 in src/D2L.Bmx/Program.cs

View workflow job for this annotation

GitHub Actions / build (windows-latest, win, bmx.exe) 

The variable 'ex' is declared but never used

Check warning on line 225 in src/D2L.Bmx/Program.cs

View workflow job for this annotation

GitHub Actions / build (macos-latest, osx, bmx) 

The variable 'ex' is declared but never used
Console.WriteLine( "Error cleaning up old files continuing..." );
alex-fang0 marked this conversation as resolved.
Show resolved Hide resolved
}
}
await next( context );
},
System.CommandLine.Invocation.MiddlewareOrder.ExceptionHandler + 1
Expand Down
Loading