![]() |
Ion Auth - Lightweight Auth System based on Redux Auth 2 - Printable Version +- CodeIgniter Forums (https://forum.codeigniter.com) +-- Forum: Archived Discussions (https://forum.codeigniter.com/forumdisplay.php?fid=20) +--- Forum: Archived Libraries & Helpers (https://forum.codeigniter.com/forumdisplay.php?fid=22) +--- Thread: Ion Auth - Lightweight Auth System based on Redux Auth 2 (/showthread.php?tid=27435) Pages:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
|
Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-08-2010 [eluser]Ben Edmunds[/eluser] Christian, When you say recover password; which step were you on? What method was being called? Thanks, Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-08-2010 [eluser]Christian Gilbert[/eluser] the controller is forgot_password there is an error in the Ion_auth library in line 734 the function is public function messages() { $_output = ''; foreach ($this->messages as $message) { $_output .= $this->message_start_delimiter . $this->ci->lang->line($message) . $this->message_end_delimiter; } return $_output; } invoked by $this->session->set_flashdata('message', $this->ion_auth->messages()); in auth controller line 165 Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-08-2010 [eluser]Ben Edmunds[/eluser] Christian, It was because their is no default value for messages and errors. I just pushed the fix for this to github. Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-08-2010 [eluser]Christian Gilbert[/eluser] it works but the flash message is empty and there is an error in rest_password: Severity: Notice Message: Undefined property: stdClass::$salt Filename: libraries/Ion_auth.php Line Number: 235 Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-08-2010 [eluser]Christian Gilbert[/eluser] I fixed the error just add $this->tables['users'].'.salt', on file ion_auth_model.php between the lines 417 and 419. Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-08-2010 [eluser]Ben Edmunds[/eluser] Thanks Christian! I just pushed that up to github. Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-09-2010 [eluser]n0xie[/eluser] I know the Controller for Ion_Auth is just some example code but I would like to point out that the way it is done now, the Controller is wide open to CSRF exploits, possibly disabling all your users. You might want to add a little check or add some documentation so that people are aware that the code is primarily meant as an example and should be validated if used in a real life site... I added a patch for those interested. Other then that, keep up the good work! Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-09-2010 [eluser]Ben Edmunds[/eluser] Thanks n0xie! I've been hoping someone would expand on the example code and this is a good step in the right direction. I'll test it and integrate it this weekend. Thanks, Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-09-2010 [eluser]matula[/eluser] Thanks for the great work, and great code! I ran across this error earlier... Code: Message: Undefined property: Ion_auth_model::$ion_auth and then this... Code: A Database Error Occurred Just FYI... Ion Auth - Lightweight Auth System based on Redux Auth 2 - El Forum - 04-10-2010 [eluser]joytopia[/eluser] Matula, I had the same issue (see my posts in this threat). Here my fixes in the model: line 651: Code: if(isset($this->ion_auth->_extra_where)) $this->db->where($this->ion_auth->_extra_where); line 844 Code: if(isset($this->ion_auth->_extra_where)) $this->db->where($this->ion_auth->_extra_where); Ben and Phil, until now it works fine with these fixes. Best regards Bernd |