[Zope3-dev] Re: Issue 363

[jürgen Kartnaller]

Done.
Florent Guillaume wrote:
Please put it in the Zope 3 collector http://www.zope.org/Collectors/Zope3-dev
Florent
jürgen Kartnaller  <[EMAIL PROTECTED]> wrote:
Is there really nobody willing to do this change or at least comment it ?
Am I completely wrong with this fix ?
Jürgen
jürgen Kartnaller wrote:
I did a check for issue 363 :
A GetoptError exception occures because of the uppercase L in the option 
list.

zope ftp only accepts lower case options l,a and d.
As I can see the implemented LIST command only uses -d option.
Am I right ?
For now this issue can be fixed like this in zope/server/ftp/server.py :
--- server.py   (revision 29853)
+++ server.py   (working copy)
@@ -206,7 +206,7 @@
opts = ()
if args.strip().startswith('-'):
try:
-opts, args = getopt(args.split(), 'lad')
+opts, args = getopt(args.split(), 'Llad')
except GetoptError:
self.reply('ERR_ARGS')
return
Someone with checkin rights can please do the cahnge.
Jürgen
___
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/ml%40nuxeo.com


___
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com

Re: [Zope3-dev] Re: Issue 363

[Florent Guillaume]

Please put it in the Zope 3 collector http://www.zope.org/Collectors/Zope3-dev

Florent

jürgen Kartnaller  <[EMAIL PROTECTED]> wrote:
> Is there really nobody willing to do this change or at least comment it ?
> 
> Am I completely wrong with this fix ?
> 
> Jürgen
> 
> jürgen Kartnaller wrote:
> > I did a check for issue 363 :
> > 
> > A GetoptError exception occures because of the uppercase L in the option 
> > list.
> > 
> > zope ftp only accepts lower case options l,a and d.
> > 
> > As I can see the implemented LIST command only uses -d option.
> > Am I right ?
> > 
> > For now this issue can be fixed like this in zope/server/ftp/server.py :
> > 
> > --- server.py   (revision 29853)
> > +++ server.py   (working copy)
> > @@ -206,7 +206,7 @@
> >  opts = ()
> >  if args.strip().startswith('-'):
> >  try:
> > -opts, args = getopt(args.split(), 'lad')
> > +opts, args = getopt(args.split(), 'Llad')
> >  except GetoptError:
> >  self.reply('ERR_ARGS')
> >  return
> > 
> > Someone with checkin rights can please do the cahnge.
> > 
> > Jürgen
> > 
> 
> ___
> Zope3-dev mailing list
> Zope3-dev@zope.org
> Unsub: http://mail.zope.org/mailman/options/zope3-dev/ml%40nuxeo.com
> 


-- 
Florent Guillaume, Nuxeo (Paris, France)   CTO, Director of R&D
+33 1 40 33 71 59   http://nuxeo.com   [EMAIL PROTECTED]
___
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com

[Zope3-dev] Re: Issue 363

[jürgen Kartnaller]

Is there really nobody willing to do this change or at least comment it ?
Am I completely wrong with this fix ?
Jürgen
jürgen Kartnaller wrote:
I did a check for issue 363 :
A GetoptError exception occures because of the uppercase L in the option 
list.

zope ftp only accepts lower case options l,a and d.
As I can see the implemented LIST command only uses -d option.
Am I right ?
For now this issue can be fixed like this in zope/server/ftp/server.py :
--- server.py   (revision 29853)
+++ server.py   (working copy)
@@ -206,7 +206,7 @@
 opts = ()
 if args.strip().startswith('-'):
 try:
-opts, args = getopt(args.split(), 'lad')
+opts, args = getopt(args.split(), 'Llad')
 except GetoptError:
 self.reply('ERR_ARGS')
 return
Someone with checkin rights can please do the cahnge.
Jürgen
___
Zope3-dev mailing list
Zope3-dev@zope.org
Unsub: http://mail.zope.org/mailman/options/zope3-dev/archive%40mail-archive.com