View Issue Details

This bug affects 1 person(s).
 6
IDProjectCategoryView StatusLast Update
13353Bug reportsAuthenticationpublic2018-02-19 15:09
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynoneSeverityminor 
Status closedResolutionfixed 
Product Version3.3.x 
Target Version3.3.xFixed in Version3.4.x 
Summary13353: Unable to update uset (mssql related)
Description

500 error when try to update user (even with empty password)

Steps To Reproduce

Try to update (or not) user (without debug) on another user.

  1. Open an update user
  2. Click save without any update
Additional Information

Only for mssql … and maybe some mssql version …

TagsNo tags attached.
Attached Files
Internal Server Error.htm (1,348 bytes)   
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"><head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>Internal Server Error</title>

<style type="text/css">
/*<![CDATA[*/
body {font-family:"Verdana";font-weight:normal;color:black;background-color:white;}
h1 { font-family:"Verdana";font-weight:normal;font-size:18pt;color:red }
h2 { font-family:"Verdana";font-weight:normal;font-size:14pt;color:maroon }
h3 {font-family:"Verdana";font-weight:bold;font-size:11pt}
p {font-family:"Verdana";font-weight:normal;color:black;font-size:9pt;margin-top: -5px}
.version {color: gray;font-size:8pt;border-top:1px solid #aaaaaa;}
/*]]>*/
</style>
</head>

<body>
<h1>Internal Server Error</h1>
<h2>CDbCommand failed to execute the SQL statement: SQLSTATE[42000]: 
[Microsoft][ODBC Driver 13 for SQL Server][SQL Server]Implicit 
conversion from data type char to binary is not allowed. Use the CONVERT
 function to run this query.</h2>
<p>
An internal error occurred while the Web server was processing your request.
Please contact the webmaster to report this problem.
</p>
<p>
Thank you.
</p>
<div class="version">
2018-02-14 13:48:51 </div>

</body></html>
Internal Server Error.htm (1,348 bytes)   
Bug heat6
Complete LimeSurvey version number (& build)3.3.0+180209
I will donate to the project if issue is resolvedNo
Browsernot relevant
Database type & versionmssql 10.50.2500
Server OS (if known)Windows 7
Webserver software & version (if known)IIS
PHP Version7.1.14 (bin from php.net)

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2018-02-14 15:28

developer   ~46452

@ollehar : you update the one_time_pw from nvarchar to binary : this seems to disallow totally any update …
unset one_time_pw when update work (i send a pull request for better detail)

But : how user can add a one_time_pw actually ? I think they just have to add it manually in SQL system. But with binary : it's more difficult.

PS : i thihnk it's related to password : but this one is string ;)

ollehar

ollehar

2018-02-14 15:32

administrator   ~46453

Which commit?

DenisChenu

DenisChenu

2018-02-14 15:43

developer   ~46454

My error : https://github.com/LimeSurvey/LimeSurvey/commit/f2ebcacc3a6117303932e6feac60b1d05772d38e

:)

Previous version:
mssql : nvarchar : https://github.com/LimeSurvey/LimeSurvey/blob/f2ebcacc3a6117303932e6feac60b1d05772d38e/installer/sql/create-mssql.sql#L533
mysql : blob : https://github.com/LimeSurvey/LimeSurvey/blob/f2ebcacc3a6117303932e6feac60b1d05772d38e/installer/sql/create-mysql.sql#L537
pgsql : bytea https://github.com/LimeSurvey/LimeSurvey/blob/f2ebcacc3a6117303932e6feac60b1d05772d38e/installer/sql/create-pgsql.sql#L542

Maybe move to string for whole and basta (more easy to use/update for admin user, and usage are really restricted)

DenisChenu

DenisChenu

2018-02-14 15:53

developer   ~46455

http://www.yiiframework.com/forum/index.php/topic/63546-sql-serverimplicit-conversion-from-data-type-varchar-to-varbinarymax-is-not-allowed-use-the-convert-function-to-run-this-query/

user70560

2018-02-15 14:28

  ~46533

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&amp;id=26484

Related Changesets

LimeSurvey: master bd408629

2018-02-15 14:28:09

user70560


Committer: GitHub Details Diff
Fixed issue 13353: Unable to update user (mssql related)

Dev: fix model for sqlsrv
Affected Issues
13353
mod - application/models/User.php Diff File

Issue History

Date Modified Username Field Change
2018-02-14 15:09 DenisChenu New Issue
2018-02-14 15:09 DenisChenu Status new => assigned
2018-02-14 15:09 DenisChenu Assigned To => DenisChenu
2018-02-14 15:09 DenisChenu File Added: Internal Server Error.htm
2018-02-14 15:10 DenisChenu Target Version => 3.3.x
2018-02-14 15:28 DenisChenu Assigned To DenisChenu => ollehar
2018-02-14 15:28 DenisChenu Status assigned => feedback
2018-02-14 15:28 DenisChenu Note Added: 46452
2018-02-14 15:32 ollehar Note Added: 46453
2018-02-14 15:37 DenisChenu Summary Unable to update password (mssql related) => Unable to update uset (mssql related)
2018-02-14 15:37 DenisChenu Description Updated
2018-02-14 15:37 DenisChenu Steps to Reproduce Updated
2018-02-14 15:40 DenisChenu Assigned To ollehar => markusfluer
2018-02-14 15:43 DenisChenu Note Added: 46454
2018-02-14 15:43 DenisChenu Status feedback => assigned
2018-02-14 15:53 DenisChenu Note Added: 46455
2018-02-15 14:28 user70560 Changeset attached => LimeSurvey master bd408629
2018-02-15 14:28 user70560 Note Added: 46533
2018-02-15 14:28 user70560 Assigned To markusfluer => user70560
2018-02-15 14:28 user70560 Resolution open => fixed
2018-02-15 14:43 DenisChenu Assigned To user70560 => DenisChenu
2018-02-15 14:43 DenisChenu Status assigned => resolved
2018-02-15 14:43 DenisChenu Fixed in Version => 3.3.x
2018-02-19 15:09 ollehar Status resolved => closed
2018-02-19 15:09 ollehar Fixed in Version 3.3.x => 3.4.x