Slogger icon indicating copy to clipboard operation
Slogger copied to clipboard

InstapaperLogger not working. Feed doesn't validate?

Open Zettt opened this issue 10 years ago • 7 comments

InstapaperLogger doesn't seem to be working either. When I run it, the plugin doesn't log any activity. I checked and this could be an Instapaper issue, or a Slogger issue.

First of all, Instapaper gives a wrong RSS feed address which looks like this:

http://www.instapaper.com//archive/rss/user-id/unique-id

Notice the double //. Gladly the RSS URL also gives an XML result without. Slogger gives the following output for ./slogger --onlyrun InstapaperLogger:

InstapaperLogger: Getting Instapaper posts for 71 accounts
/Users/me/.Slogger/plugins/instapaperlogger.rb:49:in `do_log': undefined method `each' for #<String:0x000001023d4778> (NoMethodError)
    from /Users/me/.Slogger/slogger.rb:284:in `block in run_plugins'
    from /Users/me/.Slogger/slogger.rb:271:in `each'
    from /Users/me/.Slogger/slogger.rb:271:in `run_plugins'
    from /Users/me/.Slogger/slogger.rb:447:in `<top (required)>'
    from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require'
    from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require'
    from ./slogger:18:in `<main>'

Zettt avatar Aug 24 '14 09:08 Zettt

Does your config file have square brackets around the url?

instapaper_feeds: [ 

http://www.instapaper.com//archive/rss/user-id/unique-id ]

-Brett

On 24 Aug 2014, at 4:12, Zettt wrote:

InstapaperLogger doesn't seem to be working either. When I run it, the plugin doesn't log any activity. I checked and this could be an Instapaper issue, or a Slogger issue.

First of all, Instapaper gives a wrong RSS feed address which looks like this:

http://www.instapaper.com//archive/rss/user-id/unique-id

Notice the double //. Gladly the RSS URL also gives an XML result without. Slogger gives the following output:

InstapaperLogger: Getting Instapaper posts for 71 accounts /Users/me/.Slogger/plugins/instapaperlogger.rb:49:in do_log': undefined methodeach' for #String:0x000001023d4778 (NoMethodError) from /Users/me/.Slogger/slogger.rb:284:in block in run_plugins' from /Users/me/.Slogger/slogger.rb:271:ineach' from /Users/me/.Slogger/slogger.rb:271:in run_plugins' from /Users/me/.Slogger/slogger.rb:447:in<top (required)>' from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require' from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require' from ./slogger:18:in `

'


Reply to this email directly or view it on GitHub: https://github.com/ttscoff/Slogger/issues/326

ttscoff avatar Aug 24 '14 15:08 ttscoff

It did not! The plugin still won't work though. I put the URL inside quotes before Slogger finally said something like InstapaperLogger: Getting Instapaper posts for 1 accounts. Anyway, it's not running, with or without quotes.

With quotes the output is:

/Users/me/.Slogger/plugins/instapaperlogger.rb:74:in `rescue in block in do_log': Error getting posts for http://www.instapaper.com/archive/rss/207390/WHQC3pDgVt6qyxOgjqtqfpm704 (RuntimeError)
    from /Users/me/.Slogger/plugins/instapaperlogger.rb:50:in `block in do_log'
    from /Users/me/.Slogger/plugins/instapaperlogger.rb:49:in `each'
    from /Users/me/.Slogger/plugins/instapaperlogger.rb:49:in `do_log'
    from /Users/me/.Slogger/slogger.rb:284:in `block in run_plugins'
    from /Users/me/.Slogger/slogger.rb:271:in `each'
    from /Users/me/.Slogger/slogger.rb:271:in `run_plugins'
    from /Users/me/.Slogger/slogger.rb:447:in `<top (required)>'
    from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require'
    from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require'
    from ./slogger:18:in `<main>'

Without quotes it complains there's an unexepcted : in parse from psych.rb.

Zettt avatar Aug 24 '14 21:08 Zettt

I don't use Instapaper anymore and I'm not sure I even have an account to test with. That error message is really unhelpful, though, so I might try to offer a better stack trace on that. That should help solve your issue.

-Brett

On 24 Aug 2014, at 16:37, Zettt wrote:

It did not! The plugin still won't work though. I put the URL inside quotes before Slogger finally said something like InstapaperLogger: Getting Instapaper posts for 1 accounts. Anyway, it's not running, with or without quotes.

With quotes the output is:

/Users/me/.Slogger/plugins/instapaperlogger.rb:74:in rescue in block in do_log': Error getting posts for http://www.instapaper.com/archive/rss/207390/WHQC3pDgVt6qyxOgjqtqfpm704 (RuntimeError) from /Users/me/.Slogger/plugins/instapaperlogger.rb:50:inblock in do_log' from /Users/me/.Slogger/plugins/instapaperlogger.rb:49:in each' from /Users/me/.Slogger/plugins/instapaperlogger.rb:49:indo_log' from /Users/me/.Slogger/slogger.rb:284:in block in run_plugins' from /Users/me/.Slogger/slogger.rb:271:ineach' from /Users/me/.Slogger/slogger.rb:271:in run_plugins' from /Users/me/.Slogger/slogger.rb:447:in<top (required)>' from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require' from /Users/me/.rvm/rubies/ruby-2.1.0/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require' from ./slogger:18:in `

'

Without quotes it complains there's an unexepcted : in parse from psych.rb.


Reply to this email directly or view it on GitHub: https://github.com/ttscoff/Slogger/issues/326#issuecomment-53210011

ttscoff avatar Aug 24 '14 22:08 ttscoff

Thanks. That would be lovely. I have tried several things, but, as you say, the stack trace is not that helpful.

Zettt avatar Aug 25 '14 08:08 Zettt

OK. Update. I did some more testing. I got rid of the errors referring to Psych when I put all feeds of slogger_config inside single quotes. InstapaperLogger, BlogLogger, etc. This results in the error mentioned above. Looking at instapaperlogger.rb:74 and a couple lines back, the file says:

feed_output += "\n     #{content}\n" if config['instapaper_include_content_preview'] == true

Is that correct Ruby? An if statement at the end of a line? I'm not a Ruby guy, but it looks wrong to me.

Zettt avatar Aug 25 '14 09:08 Zettt

It's correct.

On 25 Aug 2014, at 4:18, Zettt wrote:

OK. Update. I did some more testing. I got rid of the errors referring to Psych when I put all feeds of slogger_config inside single quotes. InstapaperLogger, BlogLogger, etc. This results in the error mentioned above. Looking at instapaperlogger.rb:74 and a couple lines back, the file says:

feed_output += "\n #{content}\n" if config['instapaper_include_content_preview'] == true

Is that correct Ruby? An if statement at the end of a line? I'm not a Ruby guy, but it looks wrong to me.


Reply to this email directly or view it on GitHub: https://github.com/ttscoff/Slogger/issues/326#issuecomment-53243532

ttscoff avatar Aug 25 '14 12:08 ttscoff

Instapaper is currently throwing the same set of errors for me as above...curious why this was closed when it appears the same error still occurs.

griffey avatar Feb 07 '16 01:02 griffey