Senthil Kumaran added the comment:
It is going to take a few weeks for me to get to my alerts, I will address
this as soon as I get to it .
Thanks for the triage, Irit.
On Mon, Dec 13, 2021, 12:31 AM Irit Katriel wrote:
>
> Change by Irit Katriel :
>
>
> --
Senthil Kumaran added the comment:
Pablo, we are good. The PRs were merged in open branches a while ago, and this
was tracking security releases backports.
--
___
Python tracker
<https://bugs.python.org/issue46
Senthil Kumaran added the comment:
jon, thanks for the bug report. Fixed and closed in all the branches.
--
___
Python tracker
<http://bugs.python.org/issue12
Senthil Kumaran added the comment:
Yes, single quotes should not escaped in the CGI escape. We should remove that
from the table.
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12
Senthil Kumaran added the comment:
The single quote escape was added due to Issue9061 and Issue2830. Since it was
included deliberately due to above raised issues, it is best to document it,
instead of removing it.
--
___
Python tracker
<h
Senthil Kumaran added the comment:
Just as a quick guideline. If you are talking about context manager
support for urlopen, it is available and present in 3.x but not on
2.7. And I fear, it is late to make it available on 2.7, because it is
a feature.
In any case, as Terry requested, a simple
Senthil Kumaran added the comment:
Valery, yes. I shall update 2.7 documentation with this known limitation and
3.x documentation with the example usage scenarios.
Thanks!
--
assignee: docs@python -> orsenthil
___
Python tracker
&l
Senthil Kumaran added the comment:
This is fixed in all revisions.
--
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Senthil Kumaran added the comment:
Wah Meng and I were discussing about this in a separate thread as well.
I think, it is a good idea to improve the Readme for this issue.
Wah Meng, since you already went at length to contact HP and clarify linker
issue with 64 bit and found a solution to it
Senthil Kumaran added the comment:
Wah meng - Instructions to get the _tkinter compile would be a good idea too.
Also, can you verify the same instructions would work for 3.2 version of Python?
--
keywords: +patch
Added file: http://bugs.python.org/file23221/README.patch
Senthil Kumaran added the comment:
Keith, was your python compiled with ssl? Any extra information to reproduce
this can help. (Download 2.7.2 from python.org, do a ./configure;make and
verify if this can bug can be reproduced).
--
nosy: +orsenthil
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue13024>
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Senthil Kumaran :
--
assignee: -> orsenthil
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12966>
___
___
Python-bugs-list mai
Changes by Senthil Kumaran :
--
assignee: docs@python -> orsenthil
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue13073>
___
___
Python-
Senthil Kumaran added the comment:
This is fixed the following changesets.
changeset a3f2dba93743
changeset 1ed413b52af3
changeset 277688052c5a
Thanks for the patch, Ben Hayden.
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -&g
Senthil Kumaran added the comment:
I believe, I have addressed all the comments. Closing this report.
--
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Senthil Kumaran added the comment:
Thanks for the report. This is fixed now. I hope in 3.3 I remove this old
utility functions. (real soon).
--
assignee: -> orsenthil
resolution: -> fixed
status: open -> closed
___
Python track
Senthil Kumaran added the comment:
hmm. interesting case in FreeBSD. Looks like socket.gethostname() did not
return the hostname in freebsd buildbot.
--
___
Python tracker
<http://bugs.python.org/issue13
Senthil Kumaran added the comment:
Yes, I agree. I think, it can be clarified at that point too. Because.
in 2.7 the string is being checked and in 3.3 the message_body is
checked if it's instance of bytes. But, I think, it should be
carefully worded (aligned with how other socket message
Senthil Kumaran added the comment:
Hi Ezio, I had probably overlooked this one. But It's a very interesting one
for me. Do you mind if I commit it ?
On Oct 10, 2011 7:20 PM, "Ezio Melotti" wrote:
>
> Ezio Melotti added the comment:
>
> Attached an updated patch
Senthil Kumaran added the comment:
Our discussion stemmed from this point. If you look at the change proposed,
Request class is taking a new parameter by name 'method' and it is initialized
to None:
class Request:
def __init__(self, url, data=Non
Senthil Kumaran added the comment:
Patrick,
Lots of valid points. I had not looked at the RFC spec when I mentioned about
data over request (GET) method, but was trying to derive the current
functionality of module (so that users can have a seamless experience) with
additional method=&quo
Senthil Kumaran added the comment:
Fixed in 25ff1adf5f30
--
nosy: +orsenthil
resolution: -> fixed
stage: -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Senthil Kumaran added the comment:
I have committed the patch accomodating the doc review comments which Ezio had
mentioned. At the moment, the current way seems to be most backwards compatible
one and did not want to delay it. Let's hope we get some feedback on this
method arg. T
Senthil Kumaran added the comment:
Hi Eric,
The changes suggested in the patch are good for readability, I
shall include them all. Thanks!
--
___
Python tracker
<http://bugs.python.org/issue1673
Senthil Kumaran added the comment:
Thanks for the patch, Petri and Ben.Darnell.
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12
Senthil Kumaran added the comment:
Fixed in all active branches. Thanks for the patch, Petri.
Mark, for this issue, raising ValueError from zipfile was seemingly a right
thing to do, the previous error from struct for a side effect of sending a
value lower than 1980.
--
nosy
Senthil Kumaran added the comment:
- page.encoding is a good idea.
- page.decode_content sounds definitely better than page.decode which can be
confusing as page is not a bytes object, but a file-like object.
I am thinking if an attribute to urlopen would be better? Not exactly the mode
Senthil Kumaran added the comment:
This kind of suggestion has come up before and easy fix is to add individual
schemes as the patch does. There is a number of limitations if want to make the
parser generic for any scheme. The difficult thing being the parsing behavior
and requirements as
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue13033>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
Similar issue Issue7904 and 7da7f9bfdaac wherein the accepted way to parse
x-newscheme://foo.com/stuff was added. Does the new ws:// scheme not fall under
that?
--
___
Python tracker
<http://bugs.python.
New submission from Senthil Kumaran :
This was in the docs mailing list, not translated to bug report
Paul Koning wrote:
Section 8.1.6 of the library manual talks about utcoffset(dt)-dst(dt) as the
"standard offset" and says that this should not depend on the date or +time but
o
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue13211>
___
___
Python-bugs-list mailing list
Unsubscribe:
The code is pretty straight forward and works on Linux. Is the problem
with the download? If you open the json file, is it a valid one?
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-
Senthil Kumaran added the comment:
Let's deprecate some of these (or add deprecation warnings) for
upcoming release. But, doing from xxx import * is not a recommended
way for any module, as we know that it stands to pollute the namespace
with unneccesary functions/methods. I would be ca
Senthil Kumaran added the comment:
I agree with your interpretation of the RFC. The parsing rules do not specify
any provision for inclusion of blank lines "within" the records.
However, I find that inclusion is no harm either. I checked that with a
robots.txt parser (Google webma
Senthil Kumaran added the comment:
This is fixed in all codelines. Thanks for your contribution, Jyrki Pulliainen.
--
resolution: -> fixed
status: open -> closed
versions: +Python 3.2, Python 3.3
___
Python tracker
<http://bugs.p
Senthil Kumaran added the comment:
Thanks flox for the patch.
Just the News item is added. I don't think, this requires any Docs update. One
change I had to make in the patch is to remote HTTPSHandler from __all__
because that is only conditionally available when http.client sup
Changes by Senthil Kumaran :
--
assignee: -> orsenthil
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue13323>
___
___
Python-bugs-list mai
Senthil Kumaran added the comment:
Thanks for the report, Jakub and for the patch, Jyrki
--
nosy: +orsenthil
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Senthil Kumaran added the comment:
Hi Jason & Petri,
urllib2.HTTPError never had a reason attribute. In the docs, it is mentioned
that only URLError has the reason attribute. The HTTPError sublasses URLError
and addinforurl class, but the further initialization happens only in
Senthil Kumaran added the comment:
Here is the docs patch which will help us close the issue. Addressing Eric's
last comment - I believe the what's new and News for this issue was added with
the feature, this one was Exception msg change.
--
Added file: http://bugs.
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue13425>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
Let's make it useful, that's much better instead of removing it. I am
+1 with Ezio's suggestion on this to return a list of tuples with
matching headers.
--
title: http.client.HTTPMessage.getallmatchingheaders()
Senthil Kumaran added the comment:
This is in 3.3. Thanks for the patches.
--
nosy: +orsenthil
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue11012>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
Fixed this in 2.7.
--
nosy: +orsenthil
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.org/i
Senthil Kumaran added the comment:
Updated the doc with Sandro Tosi's suggested changes in all the codelines.
--
nosy: +orsenthil
resolution: -> fixed
status: open -> closed
___
Python tracker
<http://bugs.python.
Senthil Kumaran added the comment:
The original issue was invalid.
Incorporated Michele Orrù's code style changes into the trunk and other
codelines.
--
resolution: -> fixed
status: open -> closed
title: http.server: HEAD request should not return a body -> http.serve
Senthil Kumaran added the comment:
The concern here is if the request line had something like this.
Method SP Request-URI SP HTTP-Version \r\n
The previous behavior would have resulted in
Method SP Request-URI SP HTTP-Version
That is removing only the final \r\n, whereas the
Senthil Kumaran added the comment:
Here is the patch against 3.2. Something is wrong with the mercurial at the
moment where 3.2 is shown as inactive. Once that is corrected, I shall commit
and push this to 3.2 and cpython codelines.
This bug is not present in 2.7 as different logic is
Changes by Senthil Kumaran :
--
resolution: -> fixed
stage: test needed -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Senthil Kumaran added the comment:
Some review comments. Instead of doing the inline unquote like this -
-auth = base64.b64encode(user_passwd).strip()
+auth = base64.b64encode(unquote(user_passwd)).strip()
It is better to do the explicitly above the b64 encoding step
Senthil Kumaran added the comment:
Sorry, this escaped for so long. First thing is, having different values for
HTTP_PROXY and HTTP_proxy and http_proxy is plain wrong at the user side. They
all should be same and in that case the possible problem which Kirikaza and
Neil Muller state is
Senthil Kumaran added the comment:
Joonas and Michele - The fix along with the tests is in for 2.7 line. W.r.t to
tests having it in the class level buf seems to be only easy way, for other it
seemed to be that FakeSocket and FakeConnection stuff need some major change
(and resulted in
Senthil Kumaran added the comment:
This is fixed. Thanks for your contribution.
--
resolution: -> fixed
stage: needs patch -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.o
Senthil Kumaran added the comment:
Justin,
The patch and logic is okay. We can have this is 3.3.
- I find that loop_actions as not appropriate name for the new method. It fails
to give a intuitive meaning of what is supposed to do. request_action,
request_action_continued or anything else
Senthil Kumaran added the comment:
The feature is in 3.3. I did not remove the handle_timeout method from the
Mixin class, because it might be used in the production by the existing
servers. It is not appropriate to remove methods all of sudden without
deprecation warnings and also it is not
Senthil Kumaran added the comment:
This is fixed in 3.3 now. Keeping it open for test_socketserver update. After a
ForkingServer service call, it should be asserted the collect_children routine
is run.
--
resolution: -> fixed
stage: patch review -> committed/rejected
ve
Senthil Kumaran added the comment:
Victor - Sure, I understand Issue #xxx: desc must be useful while generation
reST docs.
--
___
Python tracker
<http://bugs.python.org/issue11
New submission from Senthil Kumaran :
socketserver.ForkingMixin class has a collect_children method, that waits for
the children to exit.
But, that waits for any child process and not just the ones spawned the
socketserver alone.
try:
pid, status = os.waitpid(0, 0
Changes by Senthil Kumaran :
--
nosy: +charles-francois.natali
___
Python tracker
<http://bugs.python.org/issue12184>
___
___
Python-bugs-list mailing list
Unsub
Senthil Kumaran added the comment:
I think, if you speak to RM, you can just have this change in. Don't we get "no
commits please" email request from RM with hg or is the branch already cut?
--
___
Python tracker
<http://bugs.pyt
Senthil Kumaran added the comment:
Ori, which platform did you try to reproduce this issue. I tried in all active
codelines (cpython to all through 2.5) from hg and can't able to reproduce this
bug on Linux. If someone can reproduce, can you provide exact instruc
Senthil Kumaran added the comment:
+libpython*.so* is fine, but please don't remove the rej and orig. By
convention and practice, those files are meant to be in .ignore files to
prevent accidental checkins. Usually when the conflict occurs you are notified
immediately and you fix
Senthil Kumaran added the comment:
I strictly have not faced any problem with those rej and orig files being
present in the .hgignore. I also thought keeping the .hgignore in the repo was
for distribution and never knew that it can affect local views (If you remove
.rej and .orig your local
Senthil Kumaran added the comment:
By "for distribution", I meant that everyone who checks out ( clones)
from hg.cpython.org repository share the same .hgignore file, instead
of each person having to create their own file.
I got an impression from the earlier message that hg
Senthil Kumaran added the comment:
I tried to a line .hgignore inside .hgignore file, so that it can ignore itself
(weird concept, but wanted to try if it works) and users can make local
customization of it without any problem. This has been suggest at many places
in the Internet. I tried
Senthil Kumaran added the comment:
well, urlopen does return an file like object for socket connection which
closes itself when it goes out of scope, as you raised this bug, I think a more
explicit context manager like behavior can be tried. But I am afraid that it
would complex to implement
Senthil Kumaran added the comment:
We had support for chunked transfer encoding for POST method recently, which is
exposed via urllib2 wrapper function. PUT is not exposed via urllib2 and users
should use httplib. This feature of chunked transfer can be added to PUT by
taking the body of the
Senthil Kumaran added the comment:
Hello Stephane,
> I use HTTPConnection to simulate Apple Finder WebDAV client. When
> this WebDAV client do PUT request, it transmit data in chunked
> encoding mode and not set Content-Length HTTP field.
>
> Do you understand my context ?
I
Senthil Kumaran added the comment:
Both the changes suggested in the patch are fine. I think, it is okay to remove
the executable function. It is undocumented as it is, and have not seen any
standalone use of it. A note in NEWS can help.
--
nosy: +orsenthil
Senthil Kumaran added the comment:
I forgot completely, but in Python3, Issue5418 had already added
support to addinfourl. It is now possible to write code like
import urllib.request
with urllib.request.urlopen('http://www.python.org') as req:
res = req.read()
But yeah, unl
Senthil Kumaran added the comment:
Yes, it is a feature. Sorry that I have not paid attention to this. The Windows
(IIS) part led me to delay as I did not have any to test. Let me take this up
and see through it in 3.3.
--
___
Python tracker
Senthil Kumaran added the comment:
I see you missed the doseq parameter which is applicable in this case.
print(urllib.parse.urlencode(dictQuery, encoding='iso8859-1',doseq=True))
a=b&c=d&c=e&b=c
--
nosy: +orsenthil
resolution: -> inval
Senthil Kumaran added the comment:
Joseph, resolution applies for the bug not the process. tc. :)
--
resolution: accepted -> invalid
___
Python tracker
<http://bugs.python.org/issu
Senthil Kumaran added the comment:
Sorin, can you please open another report with more details and how some
condition in httplib breaks PyAMF. We will see through that it is fixed.
Commenting on an invalid closed issue is confusing.
--
___
Python
Senthil Kumaran added the comment:
Hi Sorin,
On Sat, Jun 25, 2011 at 07:54:24PM +, sorin wrote:
> type(message_body) is str ... even if I tried to manually force
> Python for use bytes. It seams that in 2.7 bytes are alias to str.
> Due to this the code will fail to run only on 2.
Senthil Kumaran added the comment:
Eric, sure go ahead. As I am not affected by remains of .rej and .orig (neither
have I found their usefulness (yet)), so my concerns can be stepped over. Plus,
I have found a way to ignore files via patterns in .hgrc.
I already see you and RDM proposing the
Senthil Kumaran added the comment:
Fixed in branch 2.7. Thanks Kristan. Eric, the documentation patch does not
apply to 3.1 because those functions are not present in 3.1.
--
nosy: +orsenthil
status: open -> closed
___
Python tracker
&l
Senthil Kumaran added the comment:
On Sat, Jun 25, 2011 at 08:38:04PM +, Senthil Kumaran wrote:
> As I am not affected by remains of .rej and .orig (neither have I
> found their usefulness (yet)), so my concerns can be stepped over.
Just a side-note. I wondered why is my case so,
Senthil Kumaran added the comment:
The module documentation for the SimpleHTTPServer class says that.
"""
This class serves files from the current directory and below, directly
mapping the directory structure to HTTP requests.
"""
As it is already docume
Senthil Kumaran added the comment:
On Sun, Jun 26, 2011 at 10:44:59AM +, Mark Mc Mahon wrote:
> So we need a doc patch? Should we update the older 'What's new in
> Python 2.x' for added attribs?
The Documentation patch may be applied only 2.7 and 3.2/3.3 releases
now.
Senthil Kumaran added the comment:
On Sun, Jun 26, 2011 at 03:12:30PM +, Sandro Tosi wrote:
> The patch is fine: but would you be interested in trying to write a
> unittest for select.epoll.register ? it would be really nice to
This is covered in test_epoll.py
--
nosy: +ors
Senthil Kumaran added the comment:
Eric, you mean for the whole of sqlite3 docmentation or the part in the
footnote?
--
___
Python tracker
<http://bugs.python.org/issue10
Senthil Kumaran added the comment:
Yes, I agree with you. Good Suggestion. Thanks!
--
___
Python tracker
<http://bugs.python.org/issue10403>
___
___
Python-bug
Senthil Kumaran added the comment:
As Santosa said, this was discovered and fixed in 2.7 itself, surprised that
you discovered it in any release Jon. If it was from a checkout, please update.
--
resolution: -> invalid
stage: -> committed/rejected
status: open -&g
Senthil Kumaran added the comment:
AFAIR, that capitalize part is somewhere a requirement in RFC, if the server
did not behave in proper manner, it may not be a good idea for the client to
change (or be permissive the flag).
--
nosy: +orsenthil
Senthil Kumaran added the comment:
Sorry, not "Capitalize", but the "Title" part. One can some bugs which lead to
this change in the urllib2.
--
___
Python tracker
<http://bug
Senthil Kumaran added the comment:
Fixed the indentation issues. We could not use automatic reST
footnotes because two links refer to same footnote, so left it as
such.
--
___
Python tracker
<http://bugs.python.org/issue10
Senthil Kumaran added the comment:
Hello Stefen,
Yes, I was able to reproduce the issue without the patch and at the
same time, noticed that the problem was not observed in other branches
(py3k), then saw the differences where the possible socket close
delays can happen and fixed it. As it
Senthil Kumaran added the comment:
What's new docs are usually for what is coming up new in the upcoming
releases. It is not updated once the release is done. Bug fixes and
related docs are updated in documentation.
--
___
Python tracker
I am not sure that is comprehensive enough. It is just organized into
OS specific sections and gives little details further on.
So, -1 to this, but +1 to the general idea of having detailed install
and get started documentation.
___
Python-bugs-list maili
Have fixed this. Even looking at the code, your suggestion was
correct. But even before the fix, I was not able to reproduce the bug
in Ubuntu and perhaps it was getting masked. May I ask, how did you
invoke idle that this bug surfaced and what was the platform?
Changes by Senthil Kumaran :
--
resolution: -> fixed
stage: patch review -> committed/rejected
status: open -> closed
___
Python tracker
<http://bugs.python.or
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12484>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
It should be documented that in 2.x series the file-like object does not
support context management protocol. I have added the superseder issue number,
please see the note as if you really want in 2.x, how you can have it using
contextlib and in 3.x the
New submission from Senthil Kumaran :
Update the term 'attribute' in the glossary
http://docs.python.org/dev/glossary.html#term-attribute so that the reader can
understand that the term attribute in Python can mean both 'data-attribute' and
a 'callable' meth
Changes by Senthil Kumaran :
--
nosy: +orsenthil
___
Python tracker
<http://bugs.python.org/issue12149>
___
___
Python-bugs-list mailing list
Unsubscribe:
Senthil Kumaran added the comment:
With the patch applied, test_urllib2net fails at test_ftp test case
when a valid and invalid url are presented in sequence. I think test
needs a change or a further look is needed at the patch.
--
___
Python
1 - 100 of 1788 matches
Mail list logo
