Updater is not copying over ca-bundle.crt? #164

Closed
ghost opened this Issue Aug 14, 2015 · 32 comments

Projects

None yet

10 participants

@ghost
ghost commented Aug 14, 2015

Hi,

i'm not using the updater so i can't reproduce this but i have now read from three different people that they used the updater to go from 8.0.5 to 8.1.1 (owncloud/core#17448 (comment)) and afterwards the config/ca-bundle.crt was missing.

Anyone could try to reproduce this to avoid issues with failed connections to the appstore and similar problems.

@LukasReschke
Member

Ok. That's the source why we have all the bug reports in core. Great.

We shall add an hard requirement in core and next time ensure proper testing. Thanks.

cc @karlitschek

@LukasReschke
Member

This breaks a lot. Awesome … 🚀

@LukasReschke
Member

d9b8f19

We really need some logic check that ensures that the current updater is even able to update to the next version…

@karlitschek
Member

Hmmm. Actually our documentation says that you can keep the config folder as is. That a bundle needs to be copied is not exactly intuitive or error prove.
@VicDeo Can we fix that?

@LukasReschke
Member

It's not only that bundle. - That folder is used for other purposes as well such as mimetype mappings,

That said, there is already a "fix" for this. What the problem is that the updater is just delivering updates for versions it is not capable of which will explode hard.

@LukasReschke
Member

Also related to owncloud/core#18309 - would make everything way more stable.

@ghost
ghost commented Aug 14, 2015

@LukasReschke Its not sure that the updater isn't copy over the bundle. I'm just guessing after the reports linked above and at forum/irc. This still needs to be verified.

@LukasReschke
Member

@LukasReschke Its not sure that the updater isn't copy over the bundle. I'm just guessing after the reports linked above and at forum/irc. This still needs to be verified.

It doesn't. At least not when coming from 8.0.5. Which I did on my instance :-)

@ghost
ghost commented Aug 15, 2015

Ah, sure. The files.json of the updater shipped with 8.0.5 doesn't contain the bundle:

https://github.com/owncloud/updater/blob/stable8/lib/files.json

@tech4eleven

I just tried to update via the updater in the admin section. I had 8.0.x and wanted to update to current 8.1.3. Getting a message that says:

Updating ownCloud to version 8.1.3, this may take a while.
Turned on maintenance mode
Exception: Please upload the ca-bundle.crt file into the 'config' directory.
The update was unsuccessful. Please report this issue to the ownCloud community.

What can I do? http://cloud.tech4eleven.com

@ghost
ghost commented Sep 18, 2015

This:

Please upload the ca-bundle.crt file into the 'config' directory.

until this issue here has been sorted out.

@tech4eleven

right after i posted this i figured i could try to download the own cloud zip from the website and copy over the crt file. so literally just did that and it worked. thanks all.

@ghost Unknown referenced this issue in owncloud/core Sep 21, 2015
Closed

During upgrade to 8.1.3 #19204

@MALPI
MALPI commented Sep 22, 2015

Actually after putting the zip file manually there, the upgrade process was stuck. I had to put the new binaries manually into the installation also, run occ upgrade. Still OC didn't leave maintenance mode. After setting the flag manually to false OC was at least able to start again.

Last but not least, the gallery App didn't work anymore, I had to replace it by gallery+. It crashed with a NoRoute...Exception by symfony.

Overall this upgrade was really painfull!

@karlitschek
Member

@LukasReschke where are we with moving the bundle to /core ? Can't find the pr at the moment.

@LukasReschke
Member

At the "people discussing about mimetype and whatever" state… owncloud/core#18893 🙊

@LukasReschke
Member

Merged. So now we need to ensure that the updater copies the new folder 🙈

cc @VicDeo

@23d-conception

Hello,
Not sure it's the good solution, but I have solve it simply with add an empty ca-bundle.crt file in config folder ;)

@ghost
ghost commented Sep 23, 2015

Thats definitely the worst solution. Instead follow the instructions some posts above describing where to get a valid ca-bundle.crt.

@janusbenissa

Sorry but try to update and get this error, try to find the file but it is not in the master/config ?

@janusbenissa

Thanks for the answer
But sorry i'm not a programmer and I was expect a download link so that I can upload it into the config directory (i can't make a crt file with notepad ++ otherwise I make it myself)

"Exception: Please upload the ca-bundle.crt file into the 'config' directory."

@ghost
ghost commented Oct 19, 2015

This is a download-link. Open https://raw.githubusercontent.com/owncloud/core/master/resources/config/ca-bundle.crt, save-page as -> save to config dir

@janusbenissa

Oke,, I know I was missing something :-)
Thanks, everything works now and I can use version 8.1.3

@euneuber

don't forget to make ca-bundle.crt readable to your web server ;-)

@ghost
ghost commented Oct 24, 2015

Which should be really obvious :)

@Kissaki
Kissaki commented Oct 25, 2015

Where should the file be coming from?
I checked the github repository master config folder which was linked above, with no such file.
I then downloaded 8.2.0, which does not have such a file either.
While a direct link to a ca-bundle.crt was linked above, its user content. (You can obviously check the user accounts association with owncloud, but this whole mess is less than ideal.)

@Kissaki
Kissaki commented Oct 25, 2015

The file does exist in the 8.1.3 version package.

@ghost
ghost commented Oct 25, 2015

@Kissaki For oC 8.2.0 this is not needed anymore. The file is placed within the /resources/config and is already copied over if you do a correct upgrade.

@PVince81
Contributor

Also happening when updating from 8.0.14 to 8.1.9 (using a test update server),
I can see that the file "ca-bundle.crt" does exist in the tarball, but the updater app from 8.0 doesn't seem to copy it properly.

CC @davitol

@PVince81 PVince81 added this to the 8.0.15 milestone Jul 20, 2016
@PVince81
Contributor

@VicDeo is there anything we can do with the old updater code to make it at least process this file correctly ?

@VicDeo VicDeo was assigned by PVince81 Jul 20, 2016
This was referenced Jul 20, 2016
@PVince81
Contributor

it will here #367

@PVince81 PVince81 closed this Sep 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment