Fix more filepaths

[rbdr/mobius] / hotline / files.go

diff --git a/hotline/files.go b/hotline/files.go
index 42e25db2aaec634fc3b905b081600f8c3f3f6504..dc196536d8b7abfc870c07f96ecb0dbc2a48eef5 100644 (file)
--- a/hotline/files.go
+++ b/hotline/files.go
@@ -10,24 +10,16 @@ import (
        "strings"
 )
 
        "strings"
 )
 

-func downcaseFileExtension(filename string) string {
-       splitStr := strings.Split(filename, ".")
-       ext := strings.ToLower(
-               splitStr[len(splitStr)-1],
-       )
-
-       return ext
-}
-
-func fileTypeFromFilename(fn string) fileType {
-       ft, ok := fileTypes[downcaseFileExtension(fn)]
+func fileTypeFromFilename(filename string) fileType {
+       fileExt := strings.ToLower(filepath.Ext(filename))
+       ft, ok := fileTypes[fileExt]

        if ok {
                return ft
        }
        return defaultFileType
 }
 
        if ok {
                return ft
        }
        return defaultFileType
 }
 

-func fileTypeFromInfo(info os.FileInfo) (ft fileType, err error) {
+func fileTypeFromInfo(info fs.FileInfo) (ft fileType, err error) {

        if info.IsDir() {
                ft.CreatorCode = "n/a "
                ft.TypeCode = "fldr"
        if info.IsDir() {
                ft.CreatorCode = "n/a "
                ft.TypeCode = "fldr"


@@ -38,8 +30,8 @@ func fileTypeFromInfo(info os.FileInfo) (ft fileType, err error) {
        return ft, nil
 }
 
        return ft, nil
 }
 

-func getFileNameList(filePath string) (fields []Field, err error) {
-       files, err := ioutil.ReadDir(filePath)
+func getFileNameList(path string) (fields []Field, err error) {
+       files, err := os.ReadDir(path)

        if err != nil {
                return fields, nil
        }
        if err != nil {
                return fields, nil
        }


@@ -47,25 +39,45 @@ func getFileNameList(filePath string) (fields []Field, err error) {
        for _, file := range files {
                var fnwi FileNameWithInfo
 
        for _, file := range files {
                var fnwi FileNameWithInfo
 

+               if strings.HasPrefix(file.Name(), ".") {
+                       continue
+               }
+

                fileCreator := make([]byte, 4)
 
                fileCreator := make([]byte, 4)
 

-               if file.Mode()&os.ModeSymlink != 0 {
-                       resolvedPath, err := os.Readlink(filePath + "/" + file.Name())
+               fileInfo, err := file.Info()
+               if err != nil {
+                       return fields, err
+               }
+
+               if fileInfo.Mode()&os.ModeSymlink != 0 {
+                       resolvedPath, err := os.Readlink(filepath.Join(path, file.Name()))

                        if err != nil {
                                return fields, err
                        }
 
                        if err != nil {
                                return fields, err
                        }
 

-                       rFile, err := os.Stat(filePath + "/" + resolvedPath)
+                       rFile, err := os.Stat(filepath.Join(path, resolvedPath))
+                       if errors.Is(err, os.ErrNotExist) {
+                               continue
+                       }

                        if err != nil {
                                return fields, err
                        }
 
                        if rFile.IsDir() {
                        if err != nil {
                                return fields, err
                        }
 
                        if rFile.IsDir() {

-                               dir, err := ioutil.ReadDir(filePath + "/" + file.Name())
+                               dir, err := ioutil.ReadDir(filepath.Join(path, file.Name()))

                                if err != nil {
                                        return fields, err
                                }
                                if err != nil {
                                        return fields, err
                                }

-                               binary.BigEndian.PutUint32(fnwi.FileSize[:], uint32(len(dir)))
+
+                               var c uint32
+                               for _, f := range dir {
+                                       if !strings.HasPrefix(f.Name(), ".") {
+                                               c += 1
+                                       }
+                               }
+
+                               binary.BigEndian.PutUint32(fnwi.FileSize[:], c)

                                copy(fnwi.Type[:], []byte("fldr")[:])
                                copy(fnwi.Creator[:], fileCreator[:])
                        } else {
                                copy(fnwi.Type[:], []byte("fldr")[:])
                                copy(fnwi.Creator[:], fileCreator[:])
                        } else {


@@ -75,17 +87,35 @@ func getFileNameList(filePath string) (fields []Field, err error) {
                        }
 
                } else if file.IsDir() {
                        }
 
                } else if file.IsDir() {

-                       dir, err := ioutil.ReadDir(filePath + "/" + file.Name())
+                       dir, err := ioutil.ReadDir(filepath.Join(path, file.Name()))

                        if err != nil {
                                return fields, err
                        }
                        if err != nil {
                                return fields, err
                        }

-                       binary.BigEndian.PutUint32(fnwi.FileSize[:], uint32(len(dir)))
+
+                       var c uint32
+                       for _, f := range dir {
+                               if !strings.HasPrefix(f.Name(), ".") {
+                                       c += 1
+                               }
+                       }
+
+                       binary.BigEndian.PutUint32(fnwi.FileSize[:], c)

                        copy(fnwi.Type[:], []byte("fldr")[:])
                        copy(fnwi.Creator[:], fileCreator[:])
                } else {
                        copy(fnwi.Type[:], []byte("fldr")[:])
                        copy(fnwi.Creator[:], fileCreator[:])
                } else {

-                       binary.BigEndian.PutUint32(fnwi.FileSize[:], uint32(file.Size()))
-                       copy(fnwi.Type[:], []byte(fileTypeFromFilename(file.Name()).TypeCode)[:])
-                       copy(fnwi.Creator[:], []byte(fileTypeFromFilename(file.Name()).CreatorCode)[:])
+                       // the Hotline protocol does not support fileWrapper sizes > 4GiB due to the 4 byte field size, so skip them
+                       if fileInfo.Size() > 4294967296 {
+                               continue
+                       }
+
+                       hlFile, err := newFileWrapper(&OSFileStore{}, path+"/"+file.Name(), 0)
+                       if err != nil {
+                               return nil, err
+                       }
+
+                       copy(fnwi.FileSize[:], hlFile.totalSize()[:])
+                       copy(fnwi.Type[:], hlFile.ffo.FlatFileInformationFork.TypeSignature[:])
+                       copy(fnwi.Creator[:], hlFile.ffo.FlatFileInformationFork.CreatorSignature[:])

                }
 
                strippedName := strings.Replace(file.Name(), ".incomplete", "", -1)
                }
 
                strippedName := strings.Replace(file.Name(), ".incomplete", "", -1)


@@ -134,12 +164,14 @@ func CalcTotalSize(filePath string) ([]byte, error) {
 func CalcItemCount(filePath string) ([]byte, error) {
        var itemcount uint16
        err := filepath.Walk(filePath, func(path string, info os.FileInfo, err error) error {
 func CalcItemCount(filePath string) ([]byte, error) {
        var itemcount uint16
        err := filepath.Walk(filePath, func(path string, info os.FileInfo, err error) error {

-               itemcount += 1
-

                if err != nil {
                        return err
                }
 
                if err != nil {
                        return err
                }
 

+               if !strings.HasPrefix(info.Name(), ".") {
+                       itemcount += 1
+               }
+

                return nil
        })
        if err != nil {
                return nil
        })
        if err != nil {


@@ -169,21 +201,3 @@ func EncodeFilePath(filePath string) []byte {
 
        return bytes
 }
 
        return bytes
 }

-
-const incompleteFileSuffix = ".incomplete"
-
-// effectiveFile wraps os.Open to check for the presence of a partial file transfer as a fallback
-func effectiveFile(filePath string) (*os.File, error) {
-       file, err := os.Open(filePath)
-       if err != nil && !errors.Is(err, fs.ErrNotExist) {
-               return nil, err
-       }
-
-       if errors.Is(err, fs.ErrNotExist) {
-               file, err = os.OpenFile(filePath+incompleteFileSuffix, os.O_CREATE|os.O_APPEND|os.O_WRONLY, 0644)
-               if err != nil {
-                       return nil, err
-               }
-       }
-       return file, nil
-}