Comments (3)
afaict, LookupValue: configDetails.LookupEnv
captures LookupEnv
func with configDetails
receiver. Later configDetails.Environment
value is set, this doesn't change reference to configDetails
used as receiver and as such execution of this func will use this map. But maybe I'm wrong with actual reference in use.
Do you have an explicit scenario which demonstrates such an issue ?
from compose-go.
That's correct, but only if it's not nil
to begin with. If initialized to a new map
, all's good and the project name is being picked up (even though it's added after the capture), but if it's nil
, then the project name is added to the new map, but the captured value is still nil
.
from compose-go.
I don't get it. we capture LookupEnv
func, so actually the configDetails
receiver, then when executed this function will access receiver field Environment
, even this one was set after.
Please provide an illustration example (either code or docker compose usage) to demonstrate a bug
from compose-go.
Related Issues (20)
- Regression: Decode errors are ignored
- Build error on override/merge.go line 129
- Marshalling produces invalid compose file HOT 4
- Label cannot be set to empty value in v2 HOT 1
- Support URLs as config files
- Validation doesn't catch multiple services with the same container_name HOT 1
- Bug report: `panic: interface conversion: interface {} is string, not int` HOT 1
- Added validation for container_name broken with profiles HOT 1
- Selinux key in verbose bind mount not recognized despite being in schema HOT 2
- Label not added when lables field is not defined in `docker-compose.yml` HOT 1
- Recursion in getStatementStart could be a loop HOT 1
- WithOsEnv assumes wrongly that os.Environ() returns only strings containing an equal sign HOT 4
- Version warning message causing lots of churn amongst projects with minimal benefit HOT 3
- dotenv parser shows wrong line number after inheritance syntax HOT 1
- dotenv octal escape unexpected behavior HOT 2
- templates cannot have default values containing new lines
- secrets & config mount merge violates compose spec
- Dashes in variable names <=> `environment:` block
- Panic crash if services are not indented correctly with extended compose files.
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from compose-go.