Let me fix this.

I read the docs and check the original code again, and find out that this is a feature, not a bug. It's designed purposely to load geoip and geosite files just from program dir or TROJAN_GO_LOCATION_ASSET env dir.

I think this PR can be closed.

I read the docs and check the original code again, and find out that this is a feature, not a bug. It's designed purposely to load geoip and geosite files just from program dir or TROJAN_GO_LOCATION_ASSET env dir.

I think this PR can be closed.

Hi @Loyalsoldier,

Here in https://p4gefau1t.github.io/trojan-go/basic/full-config/ said that

"geoip": "$PROGRAM_DIR$/geoip.dat",
"geosite": "$PROGRAM_DIR$/geosite.dat"

and

geoip和geosite字段指geoip和geosite数据库文件路径，默认使用程序所在目录的geoip.dat和geosite.dat。也可以通过指定环境变量TROJAN_GO_LOCATION_ASSET指定工作目录。

GeoIP and Geosite fields refer to GeoIP and GEOSITE database file paths, the default usage of the geoip.dat and geosite.dat of the directory where the program is located. You can also specify a working directory by specifying the environment variable Trojan_GO_Location_asset. 

I am not sure about the translation (from Google), but if my understanding is correct, in the doc there is a feature to allow users to specify a custom path of geoip/geosite files without using environment variable.

I am looking forward to your ideas and feedback.

Regards,
Chinese White Dolphin

Oh, my mistake.

GetAssetLocation func should changed to:

func GetAssetLocation(file string) string {
	if filepath.IsAbs(file) {
		return file
	}
	if loc := os.Getenv("TROJAN_GO_LOCATION_ASSET"); loc != "" {
		absPath, err := filepath.Abs(loc)
		if err != nil {
			log.Fatal(err)
		}
		log.Debugf("env set: TROJAN_GO_LOCATION_ASSET=%s", absPath)
		return filepath.Join(absPath, file)
	}
	return filepath.Join(GetProgramDir(), file)
}

Because there is a chance that TROJAN_GO_LOCATION_ASSET be a relative path.