Review Request 111654: add support for the discourse

[Giorgos Tsiapaliokas]

> On July 23, 2013, 1:02 p.m., Aaron J. Seigo wrote:
> > why a clone of the discourse repo on github? was this to ensure that changes that happen in upstream discourse do not cause unexpected problems? or do you have actual changes in your github clone?
> > 
> > the installation procedure needs to be recorded in the README, but i suppose i can do that once this is in.
> > 
> > one this patch is in, there will need to modifications made to the asset info API in bodega-server to include a link to the discussion forum. that implies adding a new service block to the config.json for things like the url to the discourse server.
> 
> Giorgos Tsiapaliokas wrote:
>     >why a clone of the discourse repo on github? 
>     
>     The repository is in github because at the time that I created I wasn't sure
>     if we will need it and then I just continued to use it. If github isn't good
>     for us I can move it to projects.kde.org.
>     
>     >was this to ensure that changes that happen in upstream discourse do not cause unexpected problems? 
>     >or do you have actual changes in your github clone?
>     
>     yes to both of them.
>     
>      The changes to my github clone are related to our encryption. Discourse wasn't
>     using bcrypt and since we are syncing the password hashes there was no other way in order to make
>     it possible for the users to log in.
> 
> Aaron J. Seigo wrote:
>     Ah, right, BCrypt. What does discourse use by default? I wonder if we could convince them (e.g. by a patch) to make that configurable. Hummmmm...

By default they are using Pbkdf2.

Yes we could send them a patch, the config files are already there(discourse/config/database.yml.production-sample).


- Giorgos


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111654/#review36379
-----------------------------------------------------------


On July 23, 2013, 10:20 a.m., Giorgos Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111654/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 10:20 a.m.)
> 
> 
> Review request for Bodega.
> 
> 
> Description
> -------
> 
> This patch keeps in sync the DBs of the bodega-server and discourse.
> 
> It syncs,
> 
> a. the user accounts*
> b. for each asset that has a partner with id > 1000 it create a new forum which has a topic and a post
> 
> * there is a default account for the discourse
> username: forumadmin
> password: password
> 
> 
> Diffs
> -----
> 
>   server/bodegaDbHelper 273e34f 
>   sql/discourse.plsql PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111654/diff/
> 
> 
> Testing
> -------
> 
> How to test it,
> 
> * git clone https://github.com/terietor/discourse.git
> * cd discourse
> * git checkout bodega
> * https://github.com/terietor/discourse/blob/master/docs/DEVELOPER-ADVANCED.md
> * cd bodega-server/server
> * make sql && make test-sql (your bodega user must be a superuser in order to enable the dblink extention)
> * then go back to discourse and do `bundle exec rails s`
> * go to localhost:3000
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/active/attachments/20130727/13258ae8/attachment-0001.html>