Merge pull request #128 from online-judge-tools/feature/remove-codeforces

Remove the feature to submit to Codeforces

Author: kmyk

README.md

@@ -27,7 +27,7 @@ $ pip3 install online-judge-api-client
 | [Anarchy Golf](http://golf.shinh.org/)                                         | :heavy_check_mark: | :grey_question: (same to samples) |                    |                    |                    |                    |
 | [AtCoder](https://atcoder.jp/)                                                 | :heavy_check_mark: |                    | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: |
 | [CodeChef](https://www.codechef.com/)                                          | :x: [issue](https://github.com/online-judge-tools/api-client/issues/49) |                    |                    |                    |                    |                    |
-| [Codeforces](https://codeforces.com/)                                          | :heavy_check_mark: |                    | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: |
+| [Codeforces](https://codeforces.com/)                                          | :heavy_check_mark: |                    | :heavy_check_mark: | :heavy_check_mark: | :heavy_check_mark: | :x: [issue](https://github.com/online-judge-tools/api-client/issues/127) |
 | [CS Academy](https://csacademy.com/)                                           | :heavy_check_mark: |                    |                    |                    |                    |                    |
 | [Facebook Hacker Cup](https://www.facebook.com/hackercup/)                     | :heavy_check_mark: |                    |                    |                    |                    |                    |
 | [Google Code Jam](https://codingcompetitions.withgoogle.com/codejam)           | :heavy_check_mark: |                    |                    |                    |                    |                    |

onlinejudge/_implementation/utils.py

@@ -110,7 +110,7 @@ def request(self, session: requests.Session, method: str = None, action: Optiona
         if headers is None:
             headers = {}
         if 'Referer' not in headers:
-            headers['Referer'] = url
+            headers['Referer'] = self.url
         return request(method, url, session=session, raise_for_status=raise_for_status, data=self.payload, files=self.files, headers=headers, **kwargs)
 
 

onlinejudge/service/codeforces.py

@@ -357,62 +357,12 @@ def get_available_languages(self, *, session: Optional[requests.Session] = None)
 
     def submit_code(self, code: bytes, language_id: LanguageId, *, filename: Optional[str] = None, session: Optional[requests.Session] = None) -> onlinejudge.type.Submission:
         """
-        :raises NotLoggedInError:
         :raises SubmissionError:
         """
 
-        session = session or utils.get_default_session()
-
-        # get
-        url = self.get_contest().get_url() + '/submit'
-        resp = utils.request('GET', url, session=session)
-
-        # parse
-        soup = bs4.BeautifulSoup(resp.content.decode(resp.encoding), utils.HTML_PARSER)
-        form = soup.find('form', class_='submit-form')
-        if form is None:
-            logger.error('not logged in')
-            raise NotLoggedInError
-        logger.debug('form: %s', str(form))
-
-        # make data
-        form = utils.FormSender(form, url=resp.url)
-        form.set('submittedProblemIndex', str(self.index))
-        form.set('programTypeId', language_id)
-        form.set_file('sourceFile', filename or 'code', code)
-
-        # post
-        preserved_user_agent = session.headers.get('User-Agent')
-        logger.debug('User-Agent is temporarily disabled. The old User-Agent is %s', repr(preserved_user_agent))
-        try:
-            if preserved_user_agent is not None:
-                del session.headers['User-Agent']
-            resp = form.request(session=session, raise_for_status=False)
-            try:
-                resp.raise_for_status()
-            except requests.exceptions.HTTPError as e:
-                logger.exception(e)
-                if resp.status_code == 403:
-                    logger.warning('You may use wrong User-Agent: %s', repr(session.headers.get('User-Agent')))
-                raise e
-        finally:
-            if preserved_user_agent is not None:
-                session.headers['User-Agent'] = preserved_user_agent
-
-        # result
-        if resp.url.endswith('/my'):
-            # example: https://codeforces.com/contest/598/my
-            logger.info('success: result: %s', resp.url)
-            return utils.DummySubmission(resp.url, problem=self)
-        else:
-            logger.error('failure')
-            # parse error messages
-            soup = bs4.BeautifulSoup(resp.content.decode(resp.encoding), utils.HTML_PARSER)
-            msgs = []  # type: List[str]
-            for span in soup.findAll('span', class_='error'):
-                msgs += [span.string]
-                logger.warning('Codeforces says: "%s"', span.string)
-            raise SubmissionError('it may be the "You have submitted exactly the same code before" error: ' + str(msgs))
+        msg = 'The feature to submit to Codeforces is now removed. Please read https://github.com/online-judge-tools/api-client/issues/127'
+        logger.error('%s', msg)
+        raise SubmissionError(msg)
 
     def get_url(self) -> str:
         table = {}