Feature/merge blobfuse 2.5.4

.github/copilot-instructions.md

@@ -1,6 +1,6 @@
 # Azure Storage Fuse (cloudfuse)
 
-Cloudfuse is a  FUSE filesystem driver that provides virtual filesystem backed by S3 or Azure Blob Storage. It uses libfuse (fuse3) to communicate with the Linux FUSE kernel module and implements filesystem operations using the AWS S3 or Azure Storage REST APIs.
+Cloudfuse is a FUSE filesystem driver that provides virtual filesystem backed by S3 or Azure Blob Storage. It uses libfuse (fuse3) to communicate with the Linux FUSE kernel module and implements filesystem operations using the AWS S3 or Azure Storage REST APIs.
 
 Always reference these instructions first and fallback to search or bash commands only when you encounter unexpected information that does not match the info here.
 
@@ -11,26 +11,32 @@ Always reference these instructions first and fallback to search or bash command
 **CRITICAL**: All build and test commands include specific timeout warnings. NEVER CANCEL long-running operations.
 
 - Install required dependencies:
+
   ```bash
   sudo apt update
   sudo apt install -y libfuse3-dev fuse3 gcc
   ```
 
 - Install Go 1.27.0+ (already available in most environments; version is pinned in [go.mod](../go.mod)):
+
   ```bash
   go version  # Should show 1.27.0 or higher
   ```
 
 - Build cloudfuse binary:
+
   ```bash
   ./build.sh
   ```
+
   **Timing**: ~30 seconds. NEVER CANCEL. Use timeout 120+ seconds.
 
 - Build health monitor binary:
+
   ```bash
   ./build.sh health
   ```
+
   **Timing**: ~5 seconds. Use timeout 60+ seconds.
 
 - Verify binary functionality:
@@ -42,25 +48,31 @@ Always reference these instructions first and fallback to search or bash command
 ### Testing
 
 - Run unit tests (core components only):
+
   ```bash
   go test -v -timeout=10m ./internal/... ./common/... --tags=unittest,fuse3
   ```
+
   **Timing**: ~2 minutes. NEVER CANCEL. Use timeout 15+ minutes.
 
 - Run full unit tests (some may fail without Azure credentials):
+
   ```bash
   go test -v -timeout=45m ./... --tags=unittest,fuse3
   ```
+
   **WARNING**: Expected network/credential test failures. **Timing**: ~5-10 minutes. NEVER CANCEL. Use timeout 60+ minutes.
 
 - Run linting:
+
   ```bash
   # Install golangci-lint if not available
   curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin
 
   # Run linting
   $(go env GOPATH)/bin/golangci-lint run --tests=false --build-tags fuse3 --max-issues-per-linter=0
   ```
+
   **Timing**: ~10 seconds. Use timeout 60+ seconds.
 
 - Check code formatting:
@@ -74,6 +86,7 @@ Always reference these instructions first and fallback to search or bash command
 **ALWAYS test these scenarios after making changes**:
 
 1. **Binary Creation and Basic Commands**:
+
    ```bash
    ./build.sh
    ./cloudfuse --version
@@ -82,13 +95,15 @@ Always reference these instructions first and fallback to search or bash command
    ```
 
 2. **Config Generation**:
+
    ```bash
    mkdir -p /tmp/cloudfuse-test
    ./cloudfuse gen-config --tmp-path=/tmp/cloudfuse-test --o /tmp/cloudfuse-test/config.yaml
    cat /tmp/cloudfuse-test/config.yaml
    ```
 
 3. **Health Monitor**:
+
    ```bash
    ./build.sh health
    ./cfusemon --help
@@ -197,8 +212,9 @@ $(go env GOPATH)/bin/golangci-lint run --tests=false --build-tags fuse3 --max-is
 ## Key Files to Monitor
 
 When making changes, always check these files for consistency:
+
 - `go.mod` - dependency versions
 - `main.go` - entry point
 - `build.sh` - build configuration
 - `cmd/mount.go` - core mount functionality
-- Configuration templates in `setup/` and root directory
+- Configuration templates in `setup/` and root directory

.gitignore

@@ -5,6 +5,9 @@ config*.yaml
 testcnf.yaml
 cloudfuse.log
 cloudfuse.test
+blobfuse2.log
+blobfuse2.log.*
+blobfuse2.test
 azuretest.json
 cloudfuse
 cloudfuse.exe
@@ -43,7 +46,15 @@ component/azstorage/logfile.txt
 logfile.txt
 **/logfile.txt
 **/logfile.txt.*
+# Test trace files
+*/_tmp_*.trace
+_tmp_*.trace
 # Emacs backup and auto-save files
 *~
 \#*\#
-.\#*
+.\#*
+# Python bytecode
+__pycache__/
+*.pyc
+*.pyo
+.agents

cmd/mount.go

@@ -499,6 +499,12 @@ var mountCmd = &cobra.Command{
 			return fmt.Errorf("failed to initialize logger [%s]", err.Error())
 		}
 
+		// It's best to destroy the logger before we return to the caller, as caller may abruptly exit on error and we
+		// might lose some logs in the channel which are not yet flushed to the file in case of not destroying the logger
+		defer func() {
+			_ = log.Destroy()
+		}()
+
 		if !disableVersionCheck {
 			err := VersionCheck()
 			if err != nil {
@@ -527,6 +533,13 @@ var mountCmd = &cobra.Command{
 		log.Crit("Log options: %+v", options.Logging)
 		log.Debug("Mount allowed on nonempty path : %v", options.NonEmpty)
 
+		// Mirror runtime panic/fatal stack traces to a log file (in addition to stderr, which the daemon library
+		// redirects to the per-mount .trace file). Also re-attaches the fd after in-process rotation and on SIGHUP
+		// from external rotators (logrotate, AKS Blob CSI driver, ...). Captures panics in any goroutine, including
+		// those spawned by libfuse callbacks. Called after the first log.Crit so that in syslog mode rsyslog has
+		// already created /var/log/Cloudfuse.log.
+		log.SetupCrashOutput(options.Logging.Type, options.Logging.LogFilePath)
+
 		if directIO {
 			// Direct IO is enabled, so remove the attr-cache from the pipeline
 			for i, name := range options.Components {

cmd/root.go

@@ -340,6 +340,28 @@ func parseArgs(cmdArgs []string) []string {
 	return args
 }
 
+func getTransport() *http.Transport {
+	// Prefer cloning the default transport so we inherit standard
+	// proxy/TLS settings (including HTTP(S)_PROXY and NO_PROXY).
+	if dt, ok := http.DefaultTransport.(*http.Transport); ok {
+		cp := dt.Clone()
+		cp.MaxIdleConns = 10
+		cp.IdleConnTimeout = 30 * time.Second
+		cp.DisableCompression = true // GitHub API responses are small
+		cp.DisableKeepAlives = false // Connections are reused
+		return cp
+	}
+
+	// Fallback: construct a transport that at least respects proxy env vars.
+	return &http.Transport{
+		Proxy:              http.ProxyFromEnvironment,
+		MaxIdleConns:       10,
+		IdleConnTimeout:    30 * time.Second,
+		DisableCompression: true,  // GitHub API responses are small
+		DisableKeepAlives:  false, // Connections are reused
+	}
+}
+
 // Execute : Actual command execution starts from here
 func Execute() error {
 	parsedArgs := parseArgs(os.Args)

cmd/root_test.go

@@ -97,7 +97,6 @@ func (suite *rootCmdSuite) SetupTest() {
 	if err != nil {
 		panic(fmt.Sprintf("Unable to set silent logger as default: %v", err))
 	}
-	// suite.testExecute()
 }
 
 func (suite *rootCmdSuite) cleanupTest() {