if you live in a low crime area and its not locked up like this for days on end then its a pretty good job. however, like others said, i would put ulock threw seat tube+rear wheel or chainstays+rear wheel.

second thing i would sugest would be locking up the seat and rear rack. you can get a saddle leash from bike regersity or use an old chain/cable lock and lash it threw rear rack, seat stay and the rails for the saddle.
http://www.bikeregistry.com/estore/p...f52f40603b8325

if you live in a moderate to high crime area i would upgrade your locks. either get a krypto chain lock or two ulocks, one being the new york model or the fuggedaboutit. make sure both locks are locked threw a wheel+frame+immovable object. you have a newish bike that looks pretty nice and a real thief seeing a cheap older ulock and cable lock might think its worth stealing. you can get an battery operated angle grinder at any hardware store under 100bucks and take the bike under 5mins. i like the rack you lock your bike too, and it would take probably 20mins to hacksaw the rack.

btw clean and lube your bike chain.