search

Amsterdam / schulddossier

public mirror repository
Mozilla Public License 2.0
1 stars 0 forks source link

Filestorage on Azure #279

Closed tvanoort85 closed 8 months ago

tvanoort85 commented 8 months ago

Hey Guys!

Eerste setup voor het uploaden van bestanden en het zien van bestanden! Kunnen jullie kijken wat jullie er van vinden?

Be harsh please, zodat we dit goed neer zetten zodat het bij alle andere applicaties ook te gebruiken is!

maartendekeizer commented 8 months ago

Euhm dit project gebruikt toch Flysystem als storage abstraction layer. Waarom niet gewoon de Azure adapter voor Flyststem gebruiken. Dan is de change beperkt tot wat regeltjes in de config en een andere dependency. In dit project worden tijdelijke URL's voor de storage nog in de controller gemaakt omdat de OpenStack storage adapter die ondersteuning nog niet had, maar ook dat zou je nu via Flysystem kunnen laten lopen.

tvanoort85 commented 8 months ago

Hey @maartendekeizer, Thanks voor het meedenken en de oplossing met de Azure Blob Storage Adapter had inderdaad beter geweest als deze passend was. Op dit moment heeft deze alleen geen ondersteuning voor het aanmaken van mappen, wat wel binnen de applicatie gebeurd en nodig is voor het indelen van de bestanden.

maartendekeizer commented 8 months ago

Dat komt omdat binnen Azure Blob Storage mappen niet echt bestaan. De methode kan gewoon aangeroepen worden op de adapter maar heeft geen effect, zodra er een bestand in een map wordt geplaatst bestaat de map daarna vanzelf.

ramonavic commented 8 months ago

@tvanoort85: hoeft niet per se bij deze MR, maar goed om te weten: als het goed is kun je ook de AzureBaseConfig gebruiken om de authenticatie met de DB in AzureDatabase.php te maken.

Ik heb het zelf ook nog niet doorgevoerd bij Markten, maar de authenticatie is, op de scope na, hetzelfde geloof ik.

tvanoort85 commented 8 months ago

@maartendekeizer bij het verder bekijken van de package blijkt dat deze de Microsoft/azure-storage-blob gebruikt welke in maart 2024 EOL is, zie hiervoor het volgende bericht van buitengebruikstelling van Microsoft.

Ook bij de issues van de betreffende Flysystem adapter zie je dat de vraag gesteld is hoe het verder gaat, maar het enige concrete antwoord in deze thread is dat de adapter in de huidige staat dan ook deprecated zal zijn. Hierdoor is dit, tenzij het onderhoud van de dependency door iemand opgepakt wordt, geen optie voor nu.

tongpitse commented 8 months ago

Hi Thomas,Zou mijn prive mail uit de mailing gehaald mogen worden. Met vriendelijke groet,Sieuw-Tong (Tongpi) TseVerstuurd vanaf mijn iPhoneOp 21 dec 2023 om 11:20 heeft Thomas van Oort @.***> het volgende geschreven: @maartendekeizer bij het verder bekijken van de package blijkt dat deze de Microsoft/azure-storage-blob gebruikt welke in maart 2024 EOL is, zie hiervoor het volgende bericht van buitengebruikstelling van Microsoft. Ook bij de issues van de betreffende Flysystem adapter zie je dat de vraag gesteld is hoe het verder gaat, maar het enige concrete antwoord in deze thread is dat de adapter in de huidige staat dan ook deprecated zal zijn. Hierdoor is dit, tenzij het onderhoud van de dependency door iemand opgepakt wordt, geen optie voor nu.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

tvanoort85 commented 8 months ago

Hoi @tongpitse , Helaas kan ik dat niet doen, gezien je als follower van de Github repository staat. Je kan de link Unsubscribe in de mail klikken om de emails niet meer te ontvangen of de repository op github zelf niet meer te volgen.