search

CalZoneVUB / CalZone

activation
1 stars 5 forks source link

Enhancement Database (3) #13

Open MPieter opened 10 years ago

MPieter commented 10 years ago

@nicocarraggi

Verbetering van de database-controller door de configuratie-file anders te implementeren.

Voorstel van @svdvonde was om gebruik te maken van JSON-formaat en dit dan te parsen. Zo krijgen we ook een exceptie indien het parsen niet lukt.

Ander issue is dat gevoelige informatie voor het grijpen ligt.

Note: voor latere iteraties

gaethofs commented 10 years ago

@ Note. Is het niet al vrij belangrijk dat zodra we online gaan, geen configuratie-info publiek beschikbaar is? Daarom hoeft json parsen nog niet meteen, hoewel me wel lijkt dat dit door Java zo wordt ondersteund, maar deze file moet sowieso buiten de directory. Of heb in het fout en is dit al het geval?

MPieter commented 10 years ago

Voor zover ik weet staan nu de configuration files in src/main/resources, namelijk TestdBConfig.txt en wilmadbConfig.txt.

Dus binnen de directory.

nicocarraggi commented 10 years ago

Nu dat de website online staat ben ik vrij zeker dat het niet mogelijk is om aan die .txt files te geraken. Het enige wat dan wel echt nodig is, is deze .txt files niet zichtbaar maken op github. Is dat niet genoeg dan?

MPieter commented 10 years ago

Zijt ge zeker dat ge niet rechtstreeks in den browser op die directory op de wilma server kunt, en dan zo die files accessen ?

En idd, files toevoegen aan .gitignore lijkt mij een goede oplossing.

Timbo925 commented 10 years ago

Files staan niet publiek op de Wilma server dus daar kunde normaal ni aan

gaethofs commented 10 years ago

Pas op met de mappen. Gitignore is mooi om dat buiten te houden op github maar als ge iets wat deftig kunt hacken hebt die toch meteen. Zeker niet in de eerste directory zetten. Ik kom net terug van een drukke week dus zal dit morgen eens testen dan laatk iets weten. Hoe 'onbelangrijk' het ook mag lijken dat deze informatie lijkt, het gaat nog altijd om een software engineering project waarvan men verwacht professioneel te handelen. Dus bij deze. Ik zal er eens op kijken dat alles veilig is ;).