[#MDL-6123] Unicode: Add one new parameter (xxx_original_encoding) to authentication methods

Issue Details (XML | Word | Printable)

Key:	MDL-6123
Type:	Bug
Status:	Resolved
Resolution:	Fixed
Priority:	Minor
Assignee:	Petr Skoda
Reporter:	Eloy Lafuente (stronk7)
Votes:	0
Watchers:	1

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Moodle

Unicode: Add one new parameter (xxx_original_encoding) to authentication methods

Created: 16/Jul/06 06:33 PM Updated: 21/Feb/07 04:15 PM

Return to search

Component/s:

Authentication

Affects Version/s:

1.7

Fix Version/s:

1.8

Environment:

All

Issue Links:

Dependency

This issue will be resolved by:

~~MDL-8590~~ Auth cleanup - META

~~MDL-8542~~ External db auth cleanup - META

Participants:	Eloy Lafuente (stronk7), Martin Dougiamas, Martín Langhoff and Petr Skoda
Security Level:	None
Resolved date:	21/Feb/07
Affected Branches:	MOODLE_17_STABLE
Fixed Branches:	MOODLE_18_STABLE

Description

« Hide

in order to be able to convert from external sources (ldap, db...) encoding to utf8 if Moodle is running in that mode avoiding the current utf8_decode() implemented.

Point 34 of http://docs.moodle.org/en/UTF-8_scripts

All

Comments

Change History

Version Control

Sort Order:

[ Permalink | « Hide ]

Martin Dougiamas added a comment - 17/Jul/06 03:39 PM

From Martin Langhoff (martin at catalyst.net.nz) Monday, 17 July 2006, 05:17 AM:

Yup. Makes sense to me. I am aiming to do a cleanup of auth this & next week, so I'll probably look into this. 1.6.1 target?

From Petr Skoda (skodak at centrum.cz) Monday, 17 July 2006, 03:39 PM:

my +1 for 1.7 or 1.6.2

reassigning to Martin

[ Permalink | « Hide ]

Petr Skoda added a comment - 27/Oct/06 01:37 AM

to be fixed in 1.8

[ Permalink | « Hide ]

Martín Langhoff added a comment - 27/Oct/06 04:35 AM

Hola!

If you are taking this over, similar comments to ~~MDL-5277~~.

In terms of LDAP: I did a bit of reading of the LDAP spec and the ldap code included in PHP sources to see what oyu are supposed to get. LDAP v3 says pretty much "any unicode" which could mean UTF16 or UCS or whatever. In practice, apparently all the servers use utf-8 in practice. OpenLDAP definitely does.

I am not, however, 100% confident that all PHP versions do the right thing when handling the LDAP data :-/

[ Permalink | « Hide ]

Petr Skoda added a comment - 27/Oct/06 04:51 AM

From my personal MS AD experience I know that PHP functions are returning UTF-8 too in case of v3 and default platform encoding (eg. cp1250) for v2.

I guess we could make the encoding conversion configurable so that both types can be used if needed.

[ Permalink | « Hide ]

Petr Skoda added a comment - 27/Oct/06 04:52 AM

thanks for the info!

[ Permalink | « Hide ]

Petr Skoda added a comment - 16/Feb/07 07:03 PM

fixed in ext auth db, now going to work on ldap...

[ Permalink | « Hide ]

Petr Skoda added a comment - 21/Feb/07 04:15 PM

should be fixed in cvs

All content on this web site is made available under the GNU General Public License, unless otherwise stated.