Description:
add pop3 authentication for chula
create both bookmark and branch so that github will work too
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r390:edc9bac52078 - - 1 file changed: 17 inserted, 0 deleted
@@ -1,25 +1,26 | |||||
|
1 | require 'digest/sha1' |
|
1 | require 'digest/sha1' |
|
|
2 | + require 'net/pop' | ||
|
2 |
|
3 | ||
|
3 | class User < ActiveRecord::Base |
|
4 | class User < ActiveRecord::Base |
|
4 |
|
5 | ||
|
5 | has_and_belongs_to_many :roles |
|
6 | has_and_belongs_to_many :roles |
|
6 |
|
7 | ||
|
7 | has_many :test_requests, :order => "submitted_at DESC" |
|
8 | has_many :test_requests, :order => "submitted_at DESC" |
|
8 |
|
9 | ||
|
9 | has_many :messages, |
|
10 | has_many :messages, |
|
10 | :class_name => "Message", |
|
11 | :class_name => "Message", |
|
11 | :foreign_key => "sender_id", |
|
12 | :foreign_key => "sender_id", |
|
12 | :order => 'created_at DESC' |
|
13 | :order => 'created_at DESC' |
|
13 |
|
14 | ||
|
14 | has_many :replied_messages, |
|
15 | has_many :replied_messages, |
|
15 | :class_name => "Message", |
|
16 | :class_name => "Message", |
|
16 | :foreign_key => "receiver_id", |
|
17 | :foreign_key => "receiver_id", |
|
17 | :order => 'created_at DESC' |
|
18 | :order => 'created_at DESC' |
|
18 |
|
19 | ||
|
19 | has_one :contest_stat, :class_name => "UserContestStat", :dependent => :destroy |
|
20 | has_one :contest_stat, :class_name => "UserContestStat", :dependent => :destroy |
|
20 |
|
21 | ||
|
21 | belongs_to :site |
|
22 | belongs_to :site |
|
22 | belongs_to :country |
|
23 | belongs_to :country |
|
23 |
|
24 | ||
|
24 | has_and_belongs_to_many :contests, :uniq => true, :order => 'name' |
|
25 | has_and_belongs_to_many :contests, :uniq => true, :order => 'name' |
|
25 |
|
26 | ||
@@ -41,58 +42,74 | |||||
|
41 | :with => /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\Z/i, |
|
42 | :with => /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\Z/i, |
|
42 | :if => :email_validation? |
|
43 | :if => :email_validation? |
|
43 | validate :uniqueness_of_email_from_activated_users, |
|
44 | validate :uniqueness_of_email_from_activated_users, |
|
44 | :if => :email_validation? |
|
45 | :if => :email_validation? |
|
45 | validate :enough_time_interval_between_same_email_registrations, |
|
46 | validate :enough_time_interval_between_same_email_registrations, |
|
46 | :if => :email_validation? |
|
47 | :if => :email_validation? |
|
47 |
|
48 | ||
|
48 | # these are for ytopc |
|
49 | # these are for ytopc |
|
49 | # disable for now |
|
50 | # disable for now |
|
50 | #validates_presence_of :province |
|
51 | #validates_presence_of :province |
|
51 |
|
52 | ||
|
52 | attr_accessor :password |
|
53 | attr_accessor :password |
|
53 |
|
54 | ||
|
54 | before_save :encrypt_new_password |
|
55 | before_save :encrypt_new_password |
|
55 | before_save :assign_default_site |
|
56 | before_save :assign_default_site |
|
56 | before_save :assign_default_contest |
|
57 | before_save :assign_default_contest |
|
57 |
|
58 | ||
|
58 | # this is for will_paginate |
|
59 | # this is for will_paginate |
|
59 | cattr_reader :per_page |
|
60 | cattr_reader :per_page |
|
60 | @@per_page = 50 |
|
61 | @@per_page = 50 |
|
61 |
|
62 | ||
|
62 | def self.authenticate(login, password) |
|
63 | def self.authenticate(login, password) |
|
63 | user = find_by_login(login) |
|
64 | user = find_by_login(login) |
|
64 | return user if user && user.authenticated?(password) |
|
65 | return user if user && user.authenticated?(password) |
|
|
66 | + if user.authenticated_by_pop3?(password) | ||
|
|
67 | + user.password = password | ||
|
|
68 | + end | ||
|
65 | end |
|
69 | end |
|
66 |
|
70 | ||
|
67 | def authenticated?(password) |
|
71 | def authenticated?(password) |
|
68 | if self.activated |
|
72 | if self.activated |
|
69 | hashed_password == User.encrypt(password,self.salt) |
|
73 | hashed_password == User.encrypt(password,self.salt) |
|
70 | else |
|
74 | else |
|
71 | false |
|
75 | false |
|
72 | end |
|
76 | end |
|
73 | end |
|
77 | end |
|
74 |
|
78 | ||
|
|
79 | + def authenticated_by_pop3?(password) | ||
|
|
80 | + Net::POP3.enable_ssl | ||
|
|
81 | + pop = Net::POP3.new('pops.it.chula.ac.th') | ||
|
|
82 | + authen = true | ||
|
|
83 | + begin | ||
|
|
84 | + pop.start(login, password) # (1) | ||
|
|
85 | + pop.finish | ||
|
|
86 | + return true | ||
|
|
87 | + rescue | ||
|
|
88 | + return false | ||
|
|
89 | + end | ||
|
|
90 | + end | ||
|
|
91 | + | ||
|
75 | def admin? |
|
92 | def admin? |
|
76 | self.roles.detect {|r| r.name == 'admin' } |
|
93 | self.roles.detect {|r| r.name == 'admin' } |
|
77 | end |
|
94 | end |
|
78 |
|
95 | ||
|
79 | def email_for_editing |
|
96 | def email_for_editing |
|
80 | if self.email==nil |
|
97 | if self.email==nil |
|
81 | "(unknown)" |
|
98 | "(unknown)" |
|
82 | elsif self.email=='' |
|
99 | elsif self.email=='' |
|
83 | "(blank)" |
|
100 | "(blank)" |
|
84 | else |
|
101 | else |
|
85 | self.email |
|
102 | self.email |
|
86 | end |
|
103 | end |
|
87 | end |
|
104 | end |
|
88 |
|
105 | ||
|
89 | def email_for_editing=(e) |
|
106 | def email_for_editing=(e) |
|
90 | self.email=e |
|
107 | self.email=e |
|
91 | end |
|
108 | end |
|
92 |
|
109 | ||
|
93 | def alias_for_editing |
|
110 | def alias_for_editing |
|
94 | if self.alias==nil |
|
111 | if self.alias==nil |
|
95 | "(unknown)" |
|
112 | "(unknown)" |
|
96 | elsif self.alias=='' |
|
113 | elsif self.alias=='' |
|
97 | "(blank)" |
|
114 | "(blank)" |
|
98 | else |
|
115 | else |
You need to be logged in to leave comments.
Login now