otiai10/opengraph

Only parse meta tags?

anuragdhingra opened this issue · 6 comments

anuragdhingra commented

@mbleigh @otiai10
Hey, I went through the implementation and seems like the fetch method parses title meta and link elements to create the opengraph object. Any way I could just check for meta tags?

otiai10 commented

@anuragdhingra
Is this what you want? #10

anuragdhingra commented

Yep, exactly. Thank you!

otiai10 commented

will merge. Thx for your feedback

anuragdhingra commented

@otiai10 Thank you for quick response.
Just a suggestion maybe if we could pass an optional policy to the Fetch method, it would make the method more flexible and avoid the need to use New and Parse, what do you think?
If no policy is passed maybe it can use some default policy that includes everything.

anuragdhingra commented

If it sounds okay to you, I can send a PR for the same. TIA.

otiai10 commented

Thanks. Sure you can send it.

By the way, I'm planning to release v2.0 coz I don't like the current interface of New, Fetch, and FetchWithContext as they are now.

  • 👍1