flower icon indicating copy to clipboard operation
flower copied to clipboard

More fixes, e.g making SSL transport possible.

Open etnt opened this issue 11 years ago • 4 comments

Before, tcp was hardocded in flower_datapath.erl.

etnt avatar Mar 01 '13 14:03 etnt

Hi Tobbe,

We think the setopts calls were misplaced in the flower_datapath module from the start and should be handled by the TransportMod in use (e.g. flower_tcp_transport). Thus, could you create a callback module for SSL and adjust the setopts calls accordingly? Also if you feel adventurous, I wouldn't mind having a test covering the tcp and ssl callback usage.

Let me know if you'll have a stab at this, otherwise we will. Your other changes look fine to me.

Cheers Tino

tolbrino avatar Mar 12 '13 14:03 tolbrino

Hm...we have done the transition to 80 columns already. I've pushed the code to my fork in a branch named: 80-columns So before any new changes are made you probably need to decide if you want to merge in the 80-columns branch and then we can go on from there.

The code in the branch includes all the fixes above, including the "Fixing wrong wildcard...." commit.

Cheers, Tobbe

etnt avatar Mar 13 '13 08:03 etnt

Yes, that is a big change. I'll have a look and update you soon on this.

tolbrino avatar Mar 13 '13 09:03 tolbrino

Hi again, how's it going with this 80 columns thing? It's unfortunate if we should get diverging code bases here, don't you think?

Cheers, Tobbe

etnt avatar Apr 09 '13 08:04 etnt