Re: Fwd: [PATCH] dlr-group-id : Multiple SMSCs and a DLR group

"Enver ALTIN" <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <[email protected]>
+1 from me too, although I'd probably not replace all the occurrences of
smsc_id with dlr_group_id, but my code would be a bit longer.

Thanks a lot Ben.

On 2/22/07, Alexander Malysh <[email protected]> wrote:
>
> Hi Ben,
>
> patch looks ok. I'm +1 on this one.
> Any other votes?
>
> Ben Suffolk wrote:
>
> > Guys,
> >
> > I have been using this for nearly a month now in production without
> > any problems.
> >
> > Is there any reason why it should not be committed to CVS?
> >
> > Regards
> >
> > Ben
> >
> > Begin forwarded message:
> >
> >> From: Ben Suffolk <[email protected]>
> >> Date: 29 January 2007 13:26:30 GMT
> >> To: Ben Suffolk <[email protected]>
> >> Cc: Kannel Development list <[email protected]>
> >> Subject: [PATCH] dlr-group-id : Multiple SMSCs and a DLR group
> >>
> >>> I think a simple fix would be to have another field say dlr-id,
> >>> which if not set would default to the smsc-id. This field would
> >>> then be used for the smsc name in the dlr tables. This way you can
> >>> group multiple smscs into a logical dlr group, but still have
> >>> individual control over them.
> >>>
> >>> Any thoughts on this before I have a go at implementing it?
> >>
> >> Well I got no responses in the negative to this idea, so I have
> >> implemented it.
> >>
> >> This patch allows you to add a config option to the smsc group
> >> called dlr-group-id, this lets you have several SMSCs with
> >> different id's grouped together so that they can share DLR data.
> >> i.e. in the situation that you send a message via one SMSC and get
> >> the DLR back via the other but you want to allow all the SMSCs to
> >> have different id's for control / logging purposes.
> >>
> >> If you do not specify dlr-group-id in the config is uses smsc-id as
> >> before.
> >>
> >>
> > ?
> >> Regards
> >>
> >> Ben
>
> --
> Thanks,
> Alex
>
>
>


-- 
Enver
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.